getsentry / getsentry/sentry-ruby
Expose on_rate_limit as part of the transport contract
Đang mở
@sl0thentr0py đang làm issue này rồi.
Từ ngày 31/1/2024.
Client Reports
Improvement
Ruby
- Ngôn ngữ chính
- Ruby
- Star
- 988
- Fork
- 542
- Merge trung bình
- 17 giờ 40 phút
- Pull request đã merge (30 ngày)
- 19
Mô tả
I see, thanks for the explanation of your use case.
So a few points
- we never do a retry ever on our side, a dropped event is a dropped event
- the server's rate limiting is merely interpreted as 'stop doing stuff for a while'
- technically it is breaking yes, but this is considered internal to the http transport implementation and not really part of the API contract
That said, there are a couple of options for you now
- make your own transport deriving from
Sentry::HTTPTransportand overridesend_datathere and do what you wish there - I can expose a
on_rate_limithook in the transport class similar toon_errorto make that easier so you just need to define that one method to retry
does that work?
Originally posted by @sl0thentr0py in https://github.com/getsentry/sentry-ruby/pull/2216#discussion_r1472958970
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Đánh giá
Issue này chưa được đánh giá.