Skip to content

avoid repeat captcha verify requests#74

Merged
adamjohnwright merged 1 commit intomainfrom
repeat-verify-fix
Feb 5, 2025
Merged

avoid repeat captcha verify requests#74
adamjohnwright merged 1 commit intomainfrom
repeat-verify-fix

Conversation

@GFJHogue
Copy link
Copy Markdown
Collaborator

@GFJHogue GFJHogue commented Feb 5, 2025

This addresses an edge case where /verify_captcha_page was sending 2 requests to /verify_captcha resulting in 400 Bad Request due to Cloudflare's single-use policy for Turnstile tokens.

  1. modified the javascript to only trigger form submission once.
  2. removed unnecessary turnstile .execute() call.
  3. disabled caching for verify_captcha responses to prevent strange browser behaviours.

Also removed HTTPS redirection - that's now being properly handled Apache-side.

@GFJHogue GFJHogue self-assigned this Feb 5, 2025
@adamjohnwright adamjohnwright merged commit 2044791 into main Feb 5, 2025
@GFJHogue GFJHogue deleted the repeat-verify-fix branch February 5, 2025 19:02
GFJHogue pushed a commit that referenced this pull request Apr 17, 2025
avoid repeat captcha verify requests
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.

2 participants