superfly / flyctl

Command line tools for fly.io services
https://fly.io
Apache License 2.0
1.36k stars 224 forks source link

bugfix: build upon default sentry-go transport config #3643

Closed RaghavSood closed 3 weeks ago

RaghavSood commented 3 weeks ago

Directly initializing the HTTPSyncTransport struct results in the non-exported limits map being nil, which can cause a panic in sentry-go under certain conditions

Change Summary

Fixes: #3642

What and Why: By setting the timeout on the default HTTPSyncTransport returned by NewHTTPSyncTransport, we avoid a panic within the sentry-go SDK

How:

Related to:


Documentation

dangra commented 3 weeks ago

thanks @RaghavSood !