-
Couldn't load subscription status.
- Fork 378
Deprecate ROPC flow #5355
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
Deprecate ROPC flow #5355
Conversation
src/client/Microsoft.Identity.Client/PublicClientApplication.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/ConfidentialClientApplication.cs
Outdated
Show resolved
Hide resolved
|
All of the tests using the ROPC flow will need to be migrated right? the doc says that ESTS may not issue a token |
@bgavrilMS Would it be better to just disable the tests that use the ROPC flow? |
src/client/Microsoft.Identity.Client/ConfidentialClientApplication.cs
Outdated
Show resolved
Hide resolved
tests/Microsoft.Identity.Test.Integration.netcore/HeadlessTests/OnBehalfOfTests.cs
Show resolved
Hide resolved
|
@Ugonnaak1 Can you please open a docs PR to update the following lines? We want that list to reflect product truth in MSAL.NET. It should also indicate the version in which the API deprecations happened. |
src/client/Microsoft.Identity.Client/IPublicClientApplication.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/IPublicClientApplication.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/Internal/Broker/IBroker.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/PublicClientApplication.cs
Outdated
Show resolved
Hide resolved
src/client/Microsoft.Identity.Client/PublicClientApplication.cs
Outdated
Show resolved
Hide resolved
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.
we normally do not use please. suggest removing it.
Deprecation of ROPC API due to MFA enforcement phase 2 approaching
AB#3303464