Skip to content

security: enable CSP + remove unused GoDaddy config - #129

Merged
omgitsjan merged 1 commit into
mainfrom
security/csp-remove-godaddy
Jul 25, 2026
Merged

security: enable CSP + remove unused GoDaddy config#129
omgitsjan merged 1 commit into
mainfrom
security/csp-remove-godaddy

Conversation

@omgitsjan

Copy link
Copy Markdown
Member

Summary

  • Remove unused GODADDY_API_KEY / GODADDY_API_SECRET from runtimeConfig
  • Re-enable Content-Security-Policy via nuxt-security
    • connect-src allows Cloudflare DoH (1.1.1.1, cloudflare-dns.com) + Vercel analytics
    • Scripts/styles keep 'unsafe-inline' for Nuxt UI compatibility

Test plan

  • Domain search still works (DoH not blocked by CSP)
  • No CSP errors in browser console for core UI
  • Vercel Analytics still loads if enabled

- CSP allows self + Cloudflare DoH (1.1.1.1) used by the client checker
- Drop unused GODADDY_* runtimeConfig entries
Copilot AI review requested due to automatic review settings July 25, 2026 00:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@vercel

vercel Bot commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
domain-checker Ready Ready Preview, Comment Jul 25, 2026 12:50am

@omgitsjan
omgitsjan merged commit 30f051f into main Jul 25, 2026
7 checks passed
@omgitsjan
omgitsjan deleted the security/csp-remove-godaddy branch July 25, 2026 00:52
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