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

Make the recaptcha minimum acceptable score configurable #219

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sbidoul
Copy link

@sbidoul sbidoul commented Dec 2, 2022

The recaptcha score is a value between 0 and 1.

Google recommends to analyze traffic and configure the acceptable score.

On a particular site in production, we notice a lot of spam with a score of 0.1 whereas legit submissions have a score of 0.9.

So I select a default value of 0.2, but I'm happy to set anything else that you'd feel more sensible.

@sbidoul
Copy link
Author

sbidoul commented Dec 2, 2022

cc/ @sebastienbeau

Copy link
Contributor

@sebastienbeau sebastienbeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM code review

@sebastienbeau
Copy link
Contributor

@did

Copy link

@omarie-ctc omarie-ctc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello, I need it for a customer. Could you please review this PR ? Thanks in advance.

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

Successfully merging this pull request may close these issues.

3 participants