server-auth icon indicating copy to clipboard operation
server-auth copied to clipboard

[FIX] auth_signup_verify_email: Use reCaptcha

Open Cl0ut1eR opened this issue 8 months ago • 3 comments

#774 (previous pr with same changes #775 )

Fixes an issue when using the reCaptcha addon together with auth_signup_verify_email. Currently, passwordless_signup() bypasses the reCaptcha check in the original web_auth_signup() method.

The root cause is that _verify_request_recaptcha_token() is supposed to remove the token from the request, but the same request is then used directly for user creation, resulting in: ValueError: Invalid field 'recaptcha_token_response' on model 'res.users'

Cl0ut1eR avatar Apr 03 '25 17:04 Cl0ut1eR

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days. If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

github-actions[bot] avatar Aug 03 '25 12:08 github-actions[bot]

Isn't reCaptcha an extra module?

pedrobaeza avatar Aug 12 '25 18:08 pedrobaeza

Isn't reCaptcha an extra module?

Yes but the original implementation of the method _verify_request_recaptcha_token is from base.

Cl0ut1eR avatar Aug 12 '25 18:08 Cl0ut1eR