microsoft / onefuzz

A self-hosted Fuzzing-As-A-Service platform
MIT License
2.82k stars 199 forks source link

Downgrade error to warning when retrying because of message too large #3224

Closed tevoinea closed 1 year ago

tevoinea commented 1 year ago

Summary of the Pull Request

What is this about?

PR Checklist

Info on Pull Request

What does this include?

Validation Steps Performed

How does someone test & validate?

codecov-commenter commented 1 year ago

Codecov Report

Merging #3224 (c4ca645) into main (db1e489) will increase coverage by 0.00%. The diff coverage is 66.66%.

@@           Coverage Diff           @@
##             main    #3224   +/-   ##
=======================================
  Coverage   29.54%   29.55%           
=======================================
  Files         332      332           
  Lines       39197    39199    +2     
=======================================
+ Hits        11581    11584    +3     
+ Misses      27616    27615    -1     
Impacted Files Coverage Δ
src/ApiService/ApiService/onefuzzlib/Queue.cs 40.33% <66.66%> (+1.01%) :arrow_up:

... and 1 file with indirect coverage changes