Skip to content
Snippets Groups Projects

security: improve readability of CORS handling

Merged Warren Gifford requested to merge sg/public-api-refactor into main

Created by: slimsag

The code under this if conditional would previously run if any corsOrigin was configured in the site configuration (corsOrigin != ""), and so effectively it always runs. The conditional is not needed and makes the code obtuse.

Signed-off-by: Stephen Gutekanst stephen@sourcegraph.com

Merge request reports

Approval is optional

Merged by avatar (May 5, 2025 9:07pm UTC)

Merge details

  • Changes merged into main with af63bb27.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading