opamp-go icon indicating copy to clipboard operation
opamp-go copied to clipboard

fix: incorrect switch case eval when server response code is 429

Open srikanthccv opened this issue 2 years ago • 1 comments

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.

srikanthccv avatar Aug 07 '22 13:08 srikanthccv

Codecov Report

Merging #123 (3a465c0) into main (1a47880) will increase coverage by 1.23%. The diff coverage is 100.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.

codecov[bot] avatar Aug 07 '22 13:08 codecov[bot]