Conversation
| ) | ||
|
|
||
| @staticmethod | ||
| def _override_dsn(envelope, dsn): |
There was a problem hiding this comment.
this is a bit ugly and the JS SDK doesn't do it, but maybe nice to have
|
Hey @milanboers, we appreciate that you took the time to open this PR; however, after discussing with the team, we have decided to close this PR since we already support other ways of getting events sent to multiple DSNs. First of all, by passing a custom transport to Another approach would be to use Sentry Mirror. Sentry Mirror is a proxy that provides a DSN you can send events to; the Sentry Mirror then forwards the events it receives at that DSN to multiple outgoing DSNs. By using Sentry Mirror, you would avoid having to implement a custom transport. I hope that those options solve your use case. If you have any further questions, or if we can help you in any way, please feel free too reach out to us, either by opening an issue on GitHub or by pinging us on Discord. P.S. In the future if you have a feature request, I would strongly recommend opening an issue rather than opening a PR. That way, you avoid wasting your time writing a PR if we decide against adding the feature to the SDK. |
Adds a
make_multiplexed_transportthat sends all envelopes to multiple Sentry DSNs (backends or projects).This is more or less a port of
makeMultiplexedTransportin the Javascript SDK (relevant issue).It's a common ask from customers, and there seem to be several issues about it on github/online. Would you consider adding it to the Python SDK, like it is in the Javascript SDK?
(The Javascript SDK also allows routing events based on the event content. This seems more useful for the microfrontends use case, but it can be added)
Usage:
Opened a similar PR for Java: getsentry/sentry-java#4323