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

Login adminconsent redirect_uri #2515

Open
LeonYasoon opened this issue Sep 17, 2024 · 4 comments
Open

Login adminconsent redirect_uri #2515

LeonYasoon opened this issue Sep 17, 2024 · 4 comments
Labels
teams issue issue that has been reviewed and likely teams specific

Comments

@LeonYasoon
Copy link

Hi Teams.
Not sure where to correct place to raise this issue would be, so please feel free to point me in the right direction.

When creating an adminconsent URL like https://login.microsoftonline.com/organizations/v2.0/adminconsent?client_id={{CLIENTID}}&scope={{SCOPES}}&redirect_uri={{REDIRECTURI}}&state={{STATE}} and confirming the admin consent, the user is sometimes redirect to a URI that is shaped like {{REDIRECTURI}}?admin_consent=True&tenant={{TENANTID}}&state={{state}}Sign+in+to+your+account which leads to the state being corrupted by the string "Sign in to your account" beeing appended.

@Prasad-MSFT
Copy link

Hi @LeonYasoon, thanks for raising your issue. Could you please provide more details on your issue? Any repro video would be really helpful.

@LeonYasoon
Copy link
Author

LeonYasoon commented Sep 18, 2024

Sure, here is what the flow looks like:

Open a Microsoft adminconsent login URL that is shaped like https://login.microsoftonline.com/organizations/v2.0/adminconsent?client_id={{CLIENTID}}&scope={{SCOPES}}&redirect_uri={{REDIRECTURI}}&state={{STATE}}
Screenshot 2024-09-18 at 08 55 53

Select an admin account and approve the request
Screenshot 2024-09-18 at 08 56 03

Sometimes, the redirect uri that is provided with the adminconsent URL is called with the provided state but also with the string "Sign+in+to+your+account" appended to the state query, which can be seen here in our access logs:
Screenshot 2024-09-18 at 09 09 12

@LeonYasoon
Copy link
Author

Also, please excuse me if this is not the right place to raise this request and feel free to point me in the right direction.

@TrevorJoelHarris TrevorJoelHarris added the teams issue issue that has been reviewed and likely teams specific label Sep 18, 2024
@Prasad-MSFT
Copy link

Thanks for providing the details. We are checking this with engineering team. We will inform you once we have any update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
teams issue issue that has been reviewed and likely teams specific
Projects
None yet
Development

No branches or pull requests

3 participants