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.

Sign up for GitHub

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

Jump to bottom

userguide: backport flow_id addition and tls fixes (7.0.x backports) - v1 #9855

Merged
merged 2 commits into from
Nov 22, 2023
Merged

userguide: backport flow_id addition and tls fixes (7.0.x backports) - v1 #9855

merged 2 commits into from
Nov 22, 2023

Conversation

jufajardini
Copy link
Contributor

Tickets:

Output examples compared and compatible.

@jufajardini
userguide: document flow_id, with examples
16bf5ce 
Flow_id explanation expanded from version shared by Peter Manev.

Task OISF#6445

(cherry picked from commit 58fb559)
@jufajardini
userguide: update tls not_after/not_before mentions
20f9e24 
Our tls fields not_after and not_before are actually logged as
`notafter` and `notbefore`, but were documented with the underscore.

Update the documentation, since updating the log format itself would be
a breaking change.

Task OISF#5494

(cherry picked from commit a649a92)
@victorjulien victorjulien mentioned this pull request Nov 22, 2023
Merged
@victorjulien victorjulien merged commit 20f9e24 into OISF:main-7.0.x Nov 22, 2023
37 checks passed
@victorjulien
Copy link
Member

Merged in #9863, thanks!

@jufajardini jufajardini deleted the documentation-backports/v1 branch November 27, 2023 12:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@victorjulien victorjulien victorjulien approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jufajardini @victorjulien