-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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/516/20240703/v1 #11408
next/516/20240703/v1 #11408
Conversation
Don't assume the ntlmssp version field is always present if the flag is set. Instead keep track of the offsets of the data of the various blobs and see if there is space for the version. Inspired by how Wireshark does the parsing. Bug: OISF#7121.
Ticket: OISF#6423 Not just equality, but also >3MB should work For example flow.bytes_toserver>3MB
A new workflow, "docs" has been created for only doc changes. Factor out prepare-deps into a reusable action.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #11408 +/- ##
==========================================
- Coverage 82.45% 82.44% -0.01%
==========================================
Files 938 938
Lines 248036 248044 +8
==========================================
- Hits 204523 204505 -18
- Misses 43513 43539 +26
Flags with carried forward coverage won't be shown. Click here to find out more. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Information: ERROR: QA failed on SURI_TLPW2_autofp_suri_time.
Pipeline 21353 |
Information: QA ran without warnings. Pipeline 21356 |
Staging: