-
Notifications
You must be signed in to change notification settings - Fork 448
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 support for ALTCHA #10283
Comments
Hello @asmecher, before we start implementation I wanted to confirm recommended approach with you: should this be modelled on the existing reCaptcha implementation with install-wide settings in config.inc.php and a corresponding FormValidator subclass? |
Hi Michael -- Yes, on balance I think that's best. If we are going to have several options then in my wildest dreams we'd have an interface that all options could meet, but if it's just two options (ALTCHA and ReCAPTCHA) then there's not enough benefit to justify the overhead in coding. |
I just implemented ALTCHA integration for login, register and reset password pages. You can see it on #10456. For this support to work, the To Reproduce
|
@asmecher just a heads-up that the PRs are ready to be reviewed! |
Apologies for the long wait, @Godoy0722 -- I've added some feedback here. |
Hello @asmecher . Just to let you know, I implemented the required updates if you could please take a look. I also added the altcha as a dependency for all OJS, OMP, and OPS so it can work without the minified file I added before. |
ALTCHA is a free, open source, and self-hosted alternative to reCaptcha. It's also more accessible since it doesn't require any user interaction. Github repo: https://github.com/altcha-org/altcha
The text was updated successfully, but these errors were encountered: