relay icon indicating copy to clipboard operation
relay copied to clipboard
verified publisher icon getsentry

feat(envoy): Allow to pass arbitrary http headers to upstream calls

Open dmajere opened this issue 7 months ago • 3 comments

Allow to configure and pass arbitrary headers to upstream calls.

https://github.com/getsentry/relay/issues/4712

dmajere avatar May 08 '25 00:05 dmajere

Putting to draft, waiting for feedback from @gi0baro

Dav1dde avatar May 20 '25 07:05 Dav1dde

@Dav1dde confirmed with @oioki and @gi0baro Turning it back into ready for review. Change is minor, do you need it to be in change log?

dmajere avatar May 21 '25 16:05 dmajere

@Dav1dde confirmed with @oioki and @gi0baro Turning it back into ready for review. Change is minor, do you need it to be in change log?

@dmajere while the security concern around headers was tackled by @oioki, I still don't get why we need to set headers for relay specifically. Is this only to support the legacy DSNs in the US region? Or also 'cause pops talks to frontend? If it's the latter, can't we just change the routing pops->main to talk directly to upstream releays through traffic director in place of frontend and leave relay code as it is?

gi0baro avatar May 22 '25 12:05 gi0baro