Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ACL-255] Add multi Client support #239

Merged
merged 23 commits into from
Jan 15, 2025
Merged

Conversation

tl-Roberto-Mancinelli
Copy link
Collaborator

No description provided.

@tl-Roberto-Mancinelli tl-Roberto-Mancinelli changed the title Add multiple client_id support [ACL-255] Add multi Client support Dec 20, 2024
@TrueLayer TrueLayer deleted a comment from tl-security-scanner bot Dec 20, 2024
@TrueLayer TrueLayer deleted a comment from tl-security-scanner bot Dec 20, 2024
@TrueLayer TrueLayer deleted a comment from tl-security-scanner bot Dec 20, 2024
@TrueLayer TrueLayer deleted a comment from tl-security-scanner bot Dec 20, 2024
01YG9+RQBilMp1L8MhkBCNaUS0XD9KzAnIugBwYFK4EEACOhgYkDgYYABAFRAocl
7NevcAgux1T7NXrmBiTqnMpxQmhZDa/8i/hmX7elnbUl4d78WCQbFrgrpDc6shm8
L1shuZ5G4/IbBRpAggBBHRHjQ1ZYZmgsYoszweFyUzrXWOSH349d0rmJ6gyS7/zq
2jZ884A/clvco/Zu/rSkS7Q6iWBls9w94A88Q2Phyw==
-----END EC PRIVATE KEY-----
Copy link

@tl-security-scanner tl-security-scanner bot Dec 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[FIXED] It was fixed by latest code change

@TrueLayer TrueLayer deleted a comment from tl-security-scanner bot Dec 20, 2024
README.md Outdated
Comment on lines 125 to 130
### Multiple TrueLayer Clients

Use keyed version of TrueLayer client (.NET 9.0/.NET 8.0):

```c#
.AddKeyedTrueLayer(configuration, options =>
Copy link
Contributor

@dili91 dili91 Dec 20, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! My personal preference for something like AddNamedTrueLayerClient(...), but let's keep this as it is if it's more idiomatic

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
src/TrueLayer/ApiClient.cs Outdated Show resolved Hide resolved
@tl-Roberto-Mancinelli tl-Roberto-Mancinelli marked this pull request as ready for review January 14, 2025 16:04
@tl-Roberto-Mancinelli tl-Roberto-Mancinelli requested review from a team as code owners January 14, 2025 16:04
Copy link
Contributor

@dili91 dili91 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥳

@tl-Roberto-Mancinelli tl-Roberto-Mancinelli merged commit f836fcd into main Jan 15, 2025
1 check passed
@tl-Roberto-Mancinelli tl-Roberto-Mancinelli deleted the multi-client-support branch January 15, 2025 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants