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

Template rust keywords 3195 v4 #11911

Closed

Conversation

catenacyber
Copy link
Contributor

Link to ticket: https://redmine.openinfosecfoundation.org/issues/
https://redmine.openinfosecfoundation.org/issues/3195 (generic)

Describe changes:

  • template: move keywords to rust

#11816 with review taken into account

  • new commit for allowing empty buffers
  • new SV test
  • removing usage of template-rust

SV_BRANCH=OISF/suricata-verify#2087

Ticket: 3195

Also remove unused src/tests/detect-template-buffer.c

Completes commit 4a7567b
to remove references to template-rust
Copy link

codecov bot commented Oct 8, 2024

Codecov Report

Attention: Patch coverage is 96.82540% with 2 lines in your changes missing coverage. Please review.

Project coverage is 82.64%. Comparing base (6ae5ae7) to head (4fdeb45).
Report is 8 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11911      +/-   ##
==========================================
+ Coverage   82.60%   82.64%   +0.04%     
==========================================
  Files         912      912              
  Lines      249342   249297      -45     
==========================================
+ Hits       205968   206037      +69     
+ Misses      43374    43260     -114     
Flag Coverage Δ
fuzzcorpus 60.68% <44.44%> (+0.04%) ⬆️
livemode 18.87% <7.93%> (+0.15%) ⬆️
pcap 43.95% <7.93%> (-0.13%) ⬇️
suricata-verify 62.04% <96.82%> (+0.02%) ⬆️
unittests 58.95% <7.93%> (+0.01%) ⬆️

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

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline 23057

@catenacyber catenacyber marked this pull request as draft October 9, 2024 12:59
@catenacyber
Copy link
Contributor Author

Should be rebased after #11916

@catenacyber
Copy link
Contributor Author

Rebased in #11932

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.

2 participants