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

Added support for long-running On Behalf Of flow #651

Open
wants to merge 2 commits into
base: dev
Choose a base branch
from

Conversation

HappyDump
Copy link

@HappyDump HappyDump commented May 3, 2022

Based on MSAL's new feature and the corresponding documentation fixes #644

@HappyDump
Copy link
Author

Up? This is a blocking feature for a project.

@jansenbe
Copy link
Contributor

hi @HappyDump ,

Thanks for this contribution, nice work done. We're halting the merge of this one because we need to keep authentication libraries aligned across PnP Core SDK, PnP Framework and PnP PowerShell. There's also another related issue (#660) waiting for an auth library dependency bump.

@jansenbe jansenbe self-assigned this May 30, 2022
@jansenbe jansenbe added the area: authentication 🗝 Authentication related issue or pull request label May 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: authentication 🗝 Authentication related issue or pull request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Token not refreshing with AuthenticationManager OBO
3 participants