Skip to content

fix: replace wildcard CORS subdomain check with explicit allowlist (apn-0d1oa.14)#442

Merged
paulvanbrenk merged 3 commits intomainfrom
merge/apn-0d1oa.14
Feb 26, 2026
Merged

fix: replace wildcard CORS subdomain check with explicit allowlist (apn-0d1oa.14)#442
paulvanbrenk merged 3 commits intomainfrom
merge/apn-0d1oa.14

Conversation

@paulvanbrenk
Copy link
Collaborator

Summary

  • Fixes P1 bug: CORS allowed any *.myreleasenotes.ai subdomain with credentials
  • Replaces wildcard subdomain match with an explicit origin allowlist

Issue

Closes apn-0d1oa.14

Test plan

  • 425/425 tests pass (dotnet test PatchNotes.slnx)

🤖 Merged by Refinery (ai_patch_notes merge queue processor)

…pn-0d1oa.14)

Replaced uri.Host.EndsWith('.myreleasenotes.ai') with an explicit HashSet
allowlist {app.myreleasenotes.ai, www.myreleasenotes.ai, myreleasenotes.ai}
to prevent credentialed CORS requests from dangling CNAME subdomain takeovers.
Dev overrides and AllowCredentials() preserved.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@paulvanbrenk paulvanbrenk merged commit d2d10ae into main Feb 26, 2026
4 checks passed
@paulvanbrenk paulvanbrenk deleted the merge/apn-0d1oa.14 branch February 26, 2026 03:04
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.

1 participant