Skip to content

detect/analyzer: add more details for the tcp window keyword-v3#12035

Closed
Nancyenos wants to merge 1 commit intoOISF:masterfrom
Nancyenos:detect/analyzer-add-more-details-for-the-tcp-window-keyword-v-3
Closed

detect/analyzer: add more details for the tcp window keyword-v3#12035
Nancyenos wants to merge 1 commit intoOISF:masterfrom
Nancyenos:detect/analyzer-add-more-details-for-the-tcp-window-keyword-v-3

Conversation

@Nancyenos
Copy link
Contributor

@Nancyenos Nancyenos commented Oct 25, 2024

Ticket: 6352

Make sure these boxes are checked accordingly before submitting your Pull Request -- thank you.

Contribution style:

Our Contribution agreements:

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

Describe changes:

Provide values to any of the below to override the defaults.

SV_BRANCH=OISF/suricata-verify#2107

@codecov
Copy link

codecov bot commented Oct 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.42%. Comparing base (89aa525) to head (ecaa398).
Report is 16 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #12035   +/-   ##
=======================================
  Coverage   83.42%   83.42%           
=======================================
  Files         910      910           
  Lines      257642   257649    +7     
=======================================
+ Hits       214934   214940    +6     
- Misses      42708    42709    +1     
Flag Coverage Δ
fuzzcorpus 61.56% <0.00%> (-0.01%) ⬇️
livemode 19.41% <0.00%> (-0.01%) ⬇️
pcap 44.33% <0.00%> (-0.17%) ⬇️
suricata-verify 62.80% <100.00%> (+0.03%) ⬆️
unittests 59.37% <0.00%> (-0.01%) ⬇️

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

Copy link
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 🚀

@jufajardini jufajardini added the outreachy Contributions made by Outreachy applicants label Oct 28, 2024
@inashivb
Copy link
Member

inashivb commented Nov 5, 2024

No 8.0 milestone?

@victorjulien victorjulien added this to the 8.0 milestone Nov 5, 2024
@victorjulien
Copy link
Member

Merged in #12088, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

outreachy Contributions made by Outreachy applicants

Development

Successfully merging this pull request may close these issues.

4 participants