Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Add another truncation case #3075

Merged
merged 5 commits into from
May 2, 2023

Conversation

tevoinea
Copy link
Member

@tevoinea tevoinea commented May 1, 2023

Summary of the Pull Request

What is this about?

Closes #3058

Info on Pull Request

What does this include?

An earlier PR only added truncation to the webhook code path for azure queue. This adds support for truncating and retrying for all events that support truncation.

@codecov-commenter
Copy link

Codecov Report

Merging #3075 (1feb87f) into main (6397803) will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #3075      +/-   ##
==========================================
- Coverage   31.80%   31.78%   -0.02%     
==========================================
  Files         307      307              
  Lines       36766    36782      +16     
==========================================
  Hits        11692    11692              
- Misses      25074    25090      +16     
Impacted Files Coverage Δ
src/ApiService/ApiService/onefuzzlib/Events.cs 66.01% <0.00%> (-12.15%) ⬇️

@tevoinea tevoinea enabled auto-merge (squash) May 1, 2023 16:20
@tevoinea tevoinea requested a review from chkeita May 2, 2023 17:00
@tevoinea tevoinea merged commit f3ec0e5 into microsoft:main May 2, 2023
@AdamL-Microsoft AdamL-Microsoft mentioned this pull request May 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request body too large error from onefuzzlib/Queue.cs when processing crash logs in v8.0.0
3 participants