-
Notifications
You must be signed in to change notification settings - Fork 111
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
LG-12713: Remove reCAPTCHA checkbox fallback #10454
Conversation
changelog: Internal, Spam Mitigation, Remove reCAPTCHA checkbox fallback for failed invisible assessment
recaptcha_site_key: '' | ||
recaptcha_secret_key: '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will need to make sure this renamed configuration is in place in all deployed environments before merging.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO it would be easier to not rename the credentials
recaptcha_site_key: '' | ||
recaptcha_secret_key: '' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO it would be easier to not rename the credentials
馃帿 Ticket
LG-12713
馃洜 Summary of changes
Removes the reCAPTCHA checkbox screen as a fallback when failing the invisible analysis, and replaces it with an error message informing the user that we cannot process their request.
馃摐 Testing Plan
Verify that you see an error message after submitting phone setup with a failing reCAPTCHA score:
Prerequisite: Configure score threshold in
config/application.yml
:+610491570006
)0.4
)馃憖 Screenshots