Skip to content

Remove Clerk secret key from frontend env template - #64

Merged
rajputdivyanshu81 merged 2 commits into
mainfrom
copilot/fix-clerk-secret-key-visibility
Jul 13, 2026
Merged

rajputdivyanshu81 merged 2 commits into
mainfrom
copilot/fix-clerk-secret-key-visibility

Conversation

Copilot AI commented Jul 13, 2026 •

Copy link
Copy Markdown
Contributor

The issue was a secret-management risk: CLERK_SECRET_KEY appeared in a frontend .env.example, which can mislead deployments into storing backend secrets in a broader-visibility context. This change removes that unsafe pattern and clarifies backend-only placement.

  • What changed

    • Frontend env template hardening
      • Updated apps/excelidraw-frontend/.env.example to remove CLERK_SECRET_KEY=....
      • Added an explicit note that CLERK_SECRET_KEY must be configured only in backend services (http-backend / ws-backend).
  • Why this matters

    • Prevents accidental secret exposure
      • Avoids propagating backend secret configuration into frontend environment templates.
# Before
NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=pk_test_...
CLERK_SECRET_KEY=sk_test_...

# After
NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=pk_test_...
# Set CLERK_SECRET_KEY only in backend services (http-backend/ws-backend), not in frontend env.

@vercel

vercel Bot commented Jul 13, 2026 •

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
quick-draw-docs Ready Ready Preview, Comment Jul 13, 2026 5:13am
quick-draw-excelidraw-frontend Ready Ready Preview, Comment Jul 13, 2026 5:13am
quick-draw-excelidraw-frontend-6ntt Ready Ready Preview, Comment Jul 13, 2026 5:13am

@rajputdivyanshu81
rajputdivyanshu81 marked this pull request as ready for review July 13, 2026 05:11
Copilot AI review requested due to automatic review settings July 13, 2026 05:11
Copilot AI changed the title [WIP] Fix visibility of CLERK_SECRET_KEY Remove Clerk secret key from frontend env template Jul 13, 2026
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

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.

Pull request overview

This PR hardens secret management by removing CLERK_SECRET_KEY from the frontend environment template to reduce the risk of backend secrets being mistakenly configured (and potentially exposed) in a frontend context.

Changes:

  • Removed CLERK_SECRET_KEY=... from apps/excelidraw-frontend/.env.example.
  • Added an explicit inline note indicating CLERK_SECRET_KEY must be configured only in backend services (http-backend / ws-backend).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@rajputdivyanshu81
rajputdivyanshu81 merged commit 2f16467 into main Jul 13, 2026
14 checks passed
@rajputdivyanshu81
rajputdivyanshu81 deleted the copilot/fix-clerk-secret-key-visibility branch July 13, 2026 05:13

This branch was successfully deployed

3 active deployments
Preview – quick-draw-excelidraw-frontend — 7d404d73 Deployed Jul 13, 2026 by vercel[bot]
Preview – quick-draw-excelidraw-frontend-6ntt — 7d404d73 Deployed Jul 13, 2026 by vercel[bot]
Preview – quick-draw-docs — 7d404d73 Deployed Jul 13, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clerk Issue

3 participants