[openstreetmap/openstreetmap-website] Add support for using an cloudflare turnstile widget during signup (PR #4314)

Andy Allan notifications at github.com
Wed Oct 25 10:50:04 UTC 2023


Overall looks good to me. Tests are obviously missing - we'll need some webmock to stub out the cloudflare responses, and we will want to avoid loading the JS from Cloudflare during testing too.

One bit of information from Cloudflare product pages that I noticed:

> "Managed mode: Cloudflare logic determines which users see widget"

I think that means that some users won't see the widget (the JS will just run automatically and put the result into the form). When that happens, the form label "Verify you're a human" will still show. (you can emulate this by putting a bogus `turnstile_site_key` in the config file, don't ask me how I know :smile:).

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/4314#issuecomment-1778999196
You are receiving this because you are subscribed to this thread.

Message ID: <openstreetmap/openstreetmap-website/pull/4314/c1778999196 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20231025/b6f0670d/attachment.htm>


More information about the rails-dev mailing list