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.

Sign up for GitHub

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

Jump to bottom

[Bug] Perf - MSAL stores authorities as strings and doing too many string operations. Try to use Uri instead. #3487

Closed
bgavrilMS opened this issue Jul 19, 2022 · 1 comment · Fixed by #3514
Closed

[Bug] Perf - MSAL stores authorities as strings and doing too many string operations. Try to use Uri instead. #3487

bgavrilMS opened this issue Jul 19, 2022 · 1 comment · Fixed by #3514
Labels
performance
Milestone
4.46.0

Comments

@bgavrilMS
Copy link
Member

Refactor AuthorityInfo and related code. We should not be doing so much string operations.
@MichelZ MichelZ mentioned this issue Jul 23, 2022
Merged
@pmaytak pmaytak moved this from Triage to Waiting for Code Review in MSAL Customer Trust / QM Jul 26, 2022
@pmaytak pmaytak added this to the 4.46.0 milestone Jul 26, 2022
Repository owner moved this from Waiting for Code Review to Fixed in MSAL Customer Trust / QM Jul 27, 2022
@pmaytak
Copy link
Contributor

pmaytak commented Aug 4, 2022

Released in 4.46.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
performance
Projects
MSAL Customer Trust / QM
Archived in project
Milestone
4.46.0
Development

Successfully merging a pull request may close this issue.

Refactor AuthorityInfo for Uri AzureAD/microsoft-authentication-library-for-dotnet
2 participants
@bgavrilMS @pmaytak