Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

http: fix multipart completion #8661

Closed

Conversation

catenacyber
Copy link
Contributor

Link to redmine ticket:
https://redmine.openinfosecfoundation.org/issues/5952

  • http: complete multipart until request.body-limit

Fixes #8641

suricata-verify-pr: 1109
OISF/suricata-verify#1109

As brought by commit 578f328

Ticket: OISF#5952
@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #8661 (7e6a317) into master (578f328) will decrease coverage by 0.15%.
The diff coverage is 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #8661      +/-   ##
==========================================
- Coverage   81.92%   81.77%   -0.15%     
==========================================
  Files         968      968              
  Lines      278942   278942              
==========================================
- Hits       228516   228110     -406     
- Misses      50426    50832     +406     
Flag Coverage Δ
fuzzcorpus 64.01% <100.00%> (-0.45%) ⬇️
suricata-verify 59.79% <50.00%> (+0.02%) ⬆️
unittests 63.22% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

@suricata-qa
Copy link

Information:

ERROR: QA failed on SURI_TLPW1_files_sha256.

field baseline test %
SURI_TLPW1_stats_chk
.tcp.rst 113181 144965 128.08%
SURI_TLPR1_stats_chk
.flow.memuse 530662072 1227569504 231.33%

Pipeline 12952

@catenacyber
Copy link
Contributor Author

Replaced by #8676

@catenacyber catenacyber closed this Apr 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants