opamp-go
opamp-go copied to clipboard
fix: incorrect switch case eval when server response code is 429
While working on tests for http sender I found that it doesn't respect the retry-after
for too many requests response. I believe this is a bug.
Codecov Report
Merging #123 (3a465c0) into main (1a47880) will increase coverage by
1.23%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #123 +/- ##
==========================================
+ Coverage 74.57% 75.80% +1.23%
==========================================
Files 23 23
Lines 1766 1765 -1
==========================================
+ Hits 1317 1338 +21
+ Misses 335 319 -16
+ Partials 114 108 -6
Impacted Files | Coverage Δ | |
---|---|---|
client/internal/httpsender.go | 70.96% <100.00%> (+0.45%) |
:arrow_up: |
client/internal/receivedprocessor.go | 84.28% <0.00%> (+15.00%) |
:arrow_up: |
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.