Skip to content

Setup CORS for Extension#55

Merged
hayesZach merged 6 commits intomasterfrom
feature/cors
Mar 16, 2026
Merged

Setup CORS for Extension#55
hayesZach merged 6 commits intomasterfrom
feature/cors

Conversation

@hayesZach
Copy link
Copy Markdown
Collaborator

@hayesZach hayesZach commented Mar 12, 2026

Note

Medium Risk
Introduces new CORS allowlist logic (including optional Firefox extension support) that directly affects cross-origin access to user endpoints; misconfiguration could unintentionally block or allow requests.

Overview
Adds configurable CORS handling to the HTTP server using go-chi/cors, driven by new config fields HTTP.AllowedOrigins and HTTP.AllowFirefoxExtensions.

Requests are now only allowed from explicitly configured origins, with an optional mode to permit moz-extension://... origins only when calling /api/v1/users/*; CORS is restricted to GET/OPTIONS with credentials enabled.

Written by Cursor Bugbot for commit 16683bc. This will update automatically on new commits. Configure here.

@hayesZach hayesZach self-assigned this Mar 12, 2026
@hayesZach hayesZach added the Enhancement New feature or request label Mar 12, 2026
@hayesZach hayesZach requested a review from Step7750 March 12, 2026 22:18
@hayesZach hayesZach requested a review from Step7750 March 13, 2026 20:21
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

@hayesZach hayesZach merged commit 92d8503 into master Mar 16, 2026
3 checks passed
@hayesZach hayesZach deleted the feature/cors branch March 16, 2026 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants