Skip to content

Allow Bluesky topics CORS requests - #133

Merged
haileyok merged 7 commits into
mainfrom
hailey/spaces-dpop-nonce
Aug 25, 2026
Merged

Allow Bluesky topics CORS requests#133
haileyok merged 7 commits into
mainfrom
hailey/spaces-dpop-nonce

Conversation

@haileyok

Copy link
Copy Markdown
Owner

Summary

  • Allow the x-bsky-topics request header used by the Bluesky web client.
  • Fix the app.bsky.unspecced.getTrends preflight that otherwise blocks the actual request as CORS.

Changes

  • Add x-bsky-topics to Cocoon’s explicit CORS allow-header list.
  • Extend the browser CORS regression test to cover the topics header.

Validation

  • go test ./server -run "TestCORSAllowsBrowserSpaceCredentialExchange|TestProxyResponseHeadersPreserveCocoonCORS" -count=1
  • go test ./... -count=1
  • go vet ./...
  • go test -race ./server -run "TestCORSAllowsBrowserSpaceCredentialExchange|TestProxyResponseHeadersPreserveCocoonCORS" -count=1
  • git diff --check

Review notes

  • This is a follow-up to merged PR Allow Bluesky labeler CORS requests #132.
  • The fix must be deployed before retrying the Bluesky web client; the current live preflight is expected to omit this header until deployment.

@haileyok
haileyok marked this pull request as ready for review August 25, 2026 01:18
@haileyok
haileyok merged commit aca06d3 into main Aug 25, 2026
2 checks passed
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