fix(csp): allow Cloudflare Turnstile domains for script, frame, and connect#3948
fix(csp): allow Cloudflare Turnstile domains for script, frame, and connect#3948waleedlatif1 merged 1 commit intostagingfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryMedium Risk Overview Updates both the build-time directives and the runtime-generated CSP header to permit Cloudflare Turnstile resources via Reviewed by Cursor Bugbot for commit 5f791ff. Configure here. |
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
2cd1d8b to
5f791ff
Compare
|
@greptile |
|
@cursor review |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 5f791ff. Configure here.
Greptile SummaryThis PR fixes a CSP misconfiguration that was blocking the Cloudflare Turnstile captcha widget from loading on the signup page, which was the root cause of signup failures.
Confidence Score: 5/5Safe to merge — minimal, targeted CSP fix for a legitimate Cloudflare-owned domain No P0 or P1 issues found. The change is a straightforward allowlist addition to a well-known, trusted Cloudflare domain required for Turnstile captcha functionality. Both CSP representations (build-time and runtime) are updated consistently with no drift. No files require special attention Important Files Changed
Sequence DiagramsequenceDiagram
participant Browser
participant App as Sim App
participant CF as challenges.cloudflare.com
Browser->>App: Load signup page
App-->>Browser: HTML + CSP header
Note over Browser,CF: script-src, frame-src, connect-src now allow challenges.cloudflare.com
Browser->>CF: Load Turnstile script (script-src ✓)
CF-->>Browser: Turnstile JS widget
Browser->>CF: Load Turnstile iframe (frame-src ✓)
CF-->>Browser: Challenge UI
Browser->>CF: Verification request (connect-src ✓)
CF-->>Browser: Turnstile token
Browser->>App: Submit form with token
App->>CF: Server-side token validation
CF-->>App: Validation result
Reviews (1): Last reviewed commit: "fix(csp): allow Cloudflare Turnstile dom..." | Re-trigger Greptile |
Summary
Type of Change
Testing
Tested manually
Checklist