-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Do not add v2.0 to authority endpoint if already exists #6457
Conversation
konstantin-msft
commented
Sep 11, 2023
- Do not add v2.0 to authority endpoint if already exists.
5bf6e6f
to
567efa1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Usage docs mention to always pass only domainname/tenantId
format. Curious about the usage here, it is good to have but recommend the proper usage as much as possible.
I agree that the developer shouldn't be setting the API version, but I also prefer we gracefully avoid duplication rather than throw if the7 set v2.0/ |
4d6b992
to
f30c87e
Compare
f30c87e
to
7cc5a5a
Compare