Skip to content

next/1290/80x/20260331/v1#15139

Merged
victorjulien merged 5 commits intoOISF:main-8.0.xfrom
victorjulien:next/1290/80x/20260331/v1
Apr 1, 2026
Merged

next/1290/80x/20260331/v1#15139
victorjulien merged 5 commits intoOISF:main-8.0.xfrom
victorjulien:next/1290/80x/20260331/v1

Conversation

jufajardini and others added 5 commits March 31, 2026 20:33
This is needed to generate the `--build-info` report.

Related to
Ticket OISF#7885

(cherry picked from commit 41834f0)
In some timezones, LogCustomFormatTest01 was failing as we were
specifying the wrong setting for is_dst. Instead set it to -1 so mktime
can make the decision based on the environment. Appears to fix this test
for "Brazil/East" in the summer.

Add GitHub CI tests to test this specific test with a few different
timezones.

(cherry picked from commit 468a13e)
Ticket: 8410

Do not set only request_frame_size whatever the direction

(cherry picked from commit 7b0528d)
Ticket: 8413

RFC 6455 Section 5.4 states

Control frames (see Section 5.5) MAY be injected in the middle of
a fragmented message.

Control frames are identified by opcodes where the most significant
bit of the opcode is 1.

(cherry picked from commit 2fa1005)
@victorjulien victorjulien requested review from a team, catenacyber and jasonish as code owners March 31, 2026 23:48
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.61%. Comparing base (831be60) to head (f53a5d3).
⚠️ Report is 5 commits behind head on main-8.0.x.

Additional details and impacted files
@@              Coverage Diff               @@
##           main-8.0.x   #15139      +/-   ##
==============================================
- Coverage       83.61%   83.61%   -0.01%     
==============================================
  Files            1011     1011              
  Lines          266846   266848       +2     
==============================================
- Hits           223130   223115      -15     
- Misses          43716    43733      +17     
Flag Coverage Δ
fuzzcorpus 64.02% <100.00%> (+<0.01%) ⬆️
livemode 18.72% <0.00%> (-0.05%) ⬇️
pcap 44.63% <83.33%> (-0.01%) ⬇️
suricata-verify 64.88% <83.33%> (+<0.01%) ⬆️
unittests 58.83% <14.28%> (+<0.01%) ⬆️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@suricata-qa
Copy link
Copy Markdown

Information: QA ran without warnings.

Pipeline = 30763

Copy link
Copy Markdown
Member

@inashivb inashivb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@victorjulien victorjulien merged commit f53a5d3 into OISF:main-8.0.x Apr 1, 2026
104 of 105 checks passed
@victorjulien victorjulien deleted the next/1290/80x/20260331/v1 branch April 1, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

6 participants