You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Gin's default recovery handler is clever about ignoring broken pipe errors, to avoid logging/alerting when no response will be sent because the connection has already closed. The current gin/sentry integration doesn't handle this case and we get false positive sentry alerts due to broken pipes. Integrating this check should be just a few lines of code; see https://github.com/gin-gonic/gin/blob/master/recovery.go#L45
The text was updated successfully, but these errors were encountered:
Hey, thanks for the patience. I've scheduled some time for patches/features and releasing a new version of sentry-go this and next week. I'll make sure to fix all the remaining issues till the EOD next week. Cheers!
Gin's default recovery handler is clever about ignoring broken pipe errors, to avoid logging/alerting when no response will be sent because the connection has already closed. The current gin/sentry integration doesn't handle this case and we get false positive sentry alerts due to broken pipes. Integrating this check should be just a few lines of code; see https://github.com/gin-gonic/gin/blob/master/recovery.go#L45
The text was updated successfully, but these errors were encountered: