-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Comments
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. |
@MikeAlhayek I hope you can understand what's happening |
@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". |
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 :) |
I'll try to reproduce the issue when time permits and see what is causing it. |
@Piedone @sebastienros @MikeAlhayek We are facing the same issue with the OrchardCore.ReCaptcha.Users module enabled, along with Microsoft Entra ID. The registration for new users is not working. OC Version: 2.1.3 I also troubleshot the OC main branch (tag 2.1.3) and encountered a ReCaptcha issue, resulting in a Registration Failed error. |
@ShivamGovBuilt Look at your logs for errors or debug your code further to see why did the RegisterUser method return null in your case. |
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 sayFound user using external provider and provider key.
but then redirects to /Login with anInvalid login attempt
message.Disabling ReCaptcha makes External SignIn working again.
The text was updated successfully, but these errors were encountered: