-
Notifications
You must be signed in to change notification settings - Fork 4
Feature/add cf turnstile #788
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
base: main
Are you sure you want to change the base?
Feature/add cf turnstile #788
Conversation
Added Cloudflare Turnstile as a third alternative to spam prevention.
Fixed disable and enable callbacks
added language assignment in display class before access in hcaptcha class.
fixed capitalization in class-health
|
Good day @jcedmiston We've definitely had Turnstile on our radar but had not proverbially executed on it quite yet. Definitely appreciate the PR and submission with this. We're going to review and keep the PR in mind for when we decide to roll Turnstile in with a release. It's definitely not going to be a 2.15.1 release though, but we'll adjust the details that time. |
|
@tw2113 Makes sense, I certainly don't want to mess with your roadmap. Thanks for taking the time to review. |
|
Hi @jcedmiston, thank you for the PR! I've been testing it, and while it looks good overall, I've not been able to get a form to submit using the keys provided when I set up a free Turnstyle account or the keys provided here for testing: https://developers.cloudflare.com/turnstile/troubleshooting/testing/
It's looking like Could you please check this out and confirm that the test keys from Cloudflare work? |
|
Hi @daveromsey! Yes, I have successfully used both the test keys and production keys. From your screenshot, the problem is probably upstream of When testing, you should see the Cloudflare challenge and the associated response in the source as pictured below: If you aren't seeing the challenge, it may be that the ctct-plugin-turnstile.min.js file is not being loaded properly, as that is the file that actually calls the Turnstile render method. If the challenge isn't loaded, the response token from Cloudflare won't be set. Are you getting any errors in the console related to the ctct-plugin-turnstile.min.js file? |


Added support for Cloudflare Turnstiles as a third spam prevention option for a site I help maintain. Figured it might be something others would like too!
I tried to make the formatting similar to the existing code base and added new
@sincetags for functions as what the next version would be at the time of this pull request.If I need to make any adjustments or anything else, I'm happy to.