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

userguide: document flow_id, with examples - v6 #9809

Closed
wants to merge 1 commit into from

Conversation

jufajardini
Copy link
Contributor

Flow_id explanation expanded from version shared by Peter Manev.

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

Previous PR: #9800

Changes from previous PR:

  • reword paragraph that indicates when flow_id was introduced

Results: https://suri-rtd-test.readthedocs.io/en/doc-flow-id-v6/output/eve/eve-json-format.html#flow-id

Flow_id explanation expanded from version shared by Peter Manev.

Task OISF#6445
@jufajardini jufajardini added the typo/doc update No code change : only doc or typo fixes label Nov 16, 2023
Copy link

codecov bot commented Nov 16, 2023

Codecov Report

Merging #9809 (a15fe4f) into master (2f4027c) will increase coverage by 0.00%.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #9809   +/-   ##
=======================================
  Coverage   82.37%   82.37%           
=======================================
  Files         968      968           
  Lines      273866   273866           
=======================================
+ Hits       225585   225589    +4     
+ Misses      48281    48277    -4     
Flag Coverage Δ
fuzzcorpus 64.21% <ø> (ø)
suricata-verify 60.97% <ø> (-0.01%) ⬇️
unittests 62.93% <ø> (ø)

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

@victorjulien victorjulien added this to the 8.0 milestone Nov 17, 2023
This was referenced Nov 17, 2023
@victorjulien
Copy link
Member

Merged in #9829, thanks!

@jufajardini jufajardini deleted the doc-flow-id/v6 branch November 21, 2023 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
typo/doc update No code change : only doc or typo fixes
Development

Successfully merging this pull request may close these issues.

2 participants