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

Add question/answer-based CAPTCHA script #3533

Merged
merged 3 commits into from
Feb 22, 2021
Merged

Conversation

weiss
Copy link
Member

@weiss weiss commented Feb 18, 2021

This PR is submitted on behalf of Adrien Bourmault. His commit adds a question/answer-based CAPTCHA script which is supposed to be more effective than our example script.
Example challenge: "Add 42 + 23."

I think there's two things to solve with the new script:

  • It currently relies on /bin/bash.
  • It currently only speaks English.

Adrien Bourmault and others added 2 commits February 18, 2021 19:45
Rather than replacing the original CAPTCHA script, keep it next to the
new one, for the moment.
@coveralls
Copy link

coveralls commented Feb 18, 2021

Coverage Status

Coverage decreased (-0.003%) to 35.082% when pulling 0994e79 on weiss:new-captcha into 7da033f on processone:master.

* remove various bashisms
* replace all echos with printf
+ add quotations to many variables
@prefiks
Copy link
Member

prefiks commented Feb 22, 2021

Let's merge it, we can thing about improving it when it's in tree.

@member7me
Copy link

member7me commented Apr 26, 2021

CAPTCHA realisation is't effective :( It's a pity :( Today received 2500 new spam registration.
XMPP and ejabberd are slowly killed by spam.
I should close registration, as last 2 years :(

@licaon-kter
Copy link
Contributor

@member7me Which script did you use?

No other protection?

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.

None yet

7 participants