-
Notifications
You must be signed in to change notification settings - Fork 120
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
Expose RT to broker apps #1126
Expose RT to broker apps #1126
Conversation
@rpdome Can you add more details? Perhaps a link to an ADO work item? I'm not clear on what we are trying to accomplish and why? (Broker apps aren't using MSAL right? how is this helping them?) |
They will, for device code flow. |
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.
I'm just not sure about the public API change. Other than that nits.
msal/src/main/java/com/microsoft/identity/client/IBrokerRtAccessor.java
Outdated
Show resolved
Hide resolved
msal/src/main/java/com/microsoft/identity/client/PublicClientApplication.java
Outdated
Show resolved
Hide resolved
...src/main/java/com/microsoft/identity/client/internal/BrokerClientIdRefreshTokenAccessor.java
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.
lgtm - Thanks Dome.
Allowing Broker apps to extract BRT to perform WPJ.
related PR: AzureAD/microsoft-authentication-library-common-for-android#996