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

Recaptcha and Microsoft Entra ID don't mix #17422

Open
SzymonSel opened this issue Jan 29, 2025 · 5 comments
Open

Recaptcha and Microsoft Entra ID don't mix #17422

SzymonSel opened this issue Jan 29, 2025 · 5 comments

Comments

@SzymonSel
Copy link
Member

SzymonSel commented Jan 29, 2025

I've got a weird issue. One of our sites uses both external Login (MS Entra ID) and regular login reinforced with ReCaptcha. After some recent bump in OC Libraries version (after build 18407) for whatever reason, recaptcha interferes with MS Entra ID. The logs say Found user using external provider and provider key. but then redirects to /Login with an Invalid login attempt message.

Disabling ReCaptcha makes External SignIn working again.

Image
Copy link
Contributor

We triaged this issue and set the milestone according to the priority we think is appropriate (see the docs on how we triage and prioritize issues).

This indicates when the core team may start working on it. However, if you'd like to contribute, we'd warmly welcome you to do that anytime. See our guide on contributions here.

@sebastienros
Copy link
Member

@MikeAlhayek I hope you can understand what's happening

@MikeAlhayek
Copy link
Member

@SzymonSel can you confirm that it works if you don't use reCaptcha?

It's possible that because we render the reCaptch in the form but not validate it if the user use external login "since they skip the form".

@SzymonSel
Copy link
Member Author

Yes @MikeAlhayek, precisly so! I don't know why, but there are some random situation, when it passes with ReCaptcha on :|

Maybe it's when first I do a standard login with login, then logout and try External Login. - But this doesn't help as a solution :)

@MikeAlhayek
Copy link
Member

I'll try to reproduce the issue when time permits and see what is causing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants