This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
Add Synapse-Trace-Id to access-control-allow-headers header
#14951
Assignees
Labels
T-Enhancement
New features, changes in functionality, improvements in performance, or user-facing enhancements.
Comments
DMRobertson
added
the
T-Enhancement
|
cross-posting from #14974 for visibility: @bwindels We're a bit confused about which header Synapse-Trace-Id needs to be added to. Surely it's Access-Control-Expose-Headers, so that clients can read the trace id? |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
The
Synapse-Trace-Idresponse header added in #9480 needs to be whitelisted as well so web clients using CORS can read it.Steps to reproduce
one a jaeger enabled synapse, make a request that is traced by jaeger.
Homeserver
matrix.org
Synapse Version
1.76.0rc1
Installation Method
I don't know
Database
don't know
Workers
I don't know
Platform
don't know
Configuration
No response
Relevant log output
don't have anyAnything else that would be useful to know?
No response
The text was updated successfully, but these errors were encountered: