airkjw/claude-mem
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MAESTRO: Close duplicate CORS PR #926 in favor of merged PR #917

Browse Source 
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Alex Newman
2026-02-05 18:11:51 -05:00
parent 75b9930ee4
commit e6af8d207a
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Auto Run Docs/PR-Triage/PR-Triage-03.md
+2 -1
View File
@@ -9,7 +9,8 @@ Two PRs fix the same CORS vulnerability (worker allows `Access-Control-Allow-Ori
- [x] Review and merge PR #917 (`fix: restrict CORS to localhost origins only` by @Spunky84). Files: `src/services/worker/http/middleware.ts`, `tests/worker/middleware/cors-restriction.test.ts`. Steps: (1) `gh pr checkout 917` (2) Review the CORS origin check logic — it should allow `localhost` and `127.0.0.1` origins on port 37777 only (3) Run `npm run build` to verify build passes (4) Run tests if available: check for `tests/worker/middleware/cors-restriction.test.ts` (5) If clean, rebase and merge: `gh pr merge 917 --rebase --delete-branch`
> ✅ Merged via `--admin --rebase --delete-branch`. Build passed, all 8 CORS tests passed. Code reviewed: minimal, correct origin validation with no backdoors.
- [ ] Close PR #926 (`Fix CORS misconfiguration allowing cross-site data exfiltration` by @jayvenn21) after #917 is merged. Run: `gh pr close 926 --comment "Addressed by PR #917 which restricts CORS to localhost origins with test coverage. Thank you for identifying this security issue!"`
- [x] Close PR #926 (`Fix CORS misconfiguration allowing cross-site data exfiltration` by @jayvenn21) after #917 is merged. Run: `gh pr close 926 --comment "Addressed by PR #917 which restricts CORS to localhost origins with test coverage. Thank you for identifying this security issue!"`
> ✅ Closed with thank-you comment. Duplicate of already-merged PR #917.
## XSS Vulnerability in Viewer UI