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

Next/20220727/v7 #7652

Closed
wants to merge 18 commits into from
Closed

Conversation

jufajardini and others added 18 commits July 25, 2022 20:51
alert_queue_overflow and alerts_suppressed were not being
reinitialized when there was a reload of Suricata rules, leading to
non-valid stats counters if that happened.

Bug OISF#5457
Bumps [actions/cache](https://github.com/actions/cache) from 3.0.4 to 3.0.5.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@c3f1317...0865c47)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
The image describing Suricata's sliding window had two of the "packets"
with the same text. Now they actually give the sense of a sliding
window.

This was found by Zhiyuan-liao.
@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #7652 (11e0eb9) into master (f8bf581) will increase coverage by 0.00%.
The diff coverage is 95.00%.

@@           Coverage Diff           @@
##           master    #7652   +/-   ##
=======================================
  Coverage   75.75%   75.76%           
=======================================
  Files         659      659           
  Lines      185743   185748    +5     
=======================================
+ Hits       140713   140724   +11     
+ Misses      45030    45024    -6     
Flag Coverage Δ
fuzzcorpus 60.18% <85.00%> (+0.03%) ⬆️
suricata-verify 52.50% <85.00%> (+0.06%) ⬆️
unittests 60.70% <0.00%> (-0.02%) ⬇️

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

@victorjulien victorjulien mentioned this pull request Jul 28, 2022
@victorjulien
Copy link
Member Author

Replaced by #7653

@victorjulien victorjulien deleted the next/20220727/v7 branch July 17, 2023 11:55
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.

3 participants