Skip to content

Smtp frames/v22#11229

Closed
victorjulien wants to merge 12 commits intoOISF:masterfrom
victorjulien:smtp-frames/v22
Closed

Smtp frames/v22#11229
victorjulien wants to merge 12 commits intoOISF:masterfrom
victorjulien:smtp-frames/v22

Conversation

@victorjulien
Copy link
Member

@victorjulien victorjulien commented Jun 4, 2024

'dir' was too generic, so indicate it's about the app-layer update direction.
'dir' was too generic, so indicate it's about the app-layer update direction.
Getter for the most recent frame with unknown length (-1).
AppLayerFrameGetLastOpenByType: Returns the most recent frame with a type
with unknown length (-1).

Check if type is globally enabled first.
Make sure to only scan the data when the app layer has been updated
as well.

Ticket: OISF#6718.
Add new flags to trigger FLOW_TS_APP_UPDATED/FLOW_TC_APP_UPDATED flags
to be set for the next packet in the relevant direction.

This allows for app relevant work to be done in the next packet in our
direction.
Inspect frames in the correct direction after they have been created.
There will generally not be an opposing direction to handle
the app update.
Adds the following frames:

  command_line
  data
  response_line

The *_line frames are per line, so in multi-line responses each line
will have it's own frame.

Ticket: OISF#4905.
@victorjulien victorjulien requested a review from jufajardini as a code owner June 4, 2024 10:32
@victorjulien victorjulien mentioned this pull request Jun 4, 2024
@codecov
Copy link

codecov bot commented Jun 4, 2024

Codecov Report

Attention: Patch coverage is 97.91667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 82.98%. Comparing base (a10c1f1) to head (fb0074f).

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #11229      +/-   ##
==========================================
+ Coverage   82.96%   82.98%   +0.01%     
==========================================
  Files         942      942              
  Lines      249246   249358     +112     
==========================================
+ Hits       206789   206921     +132     
+ Misses      42457    42437      -20     
Flag Coverage Δ
fuzzcorpus 61.10% <69.44%> (+<0.01%) ⬆️
livemode 18.79% <7.69%> (-0.01%) ⬇️
pcap 44.32% <74.12%> (+0.03%) ⬆️
suricata-verify 61.66% <97.20%> (+0.03%) ⬆️
unittests 60.50% <41.66%> (-0.02%) ⬇️

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

Copy link
Member

@jasonish jasonish left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doc updates look good.

@victorjulien victorjulien added this to the 8.0 milestone Jun 4, 2024
@victorjulien
Copy link
Member Author

Merged in #11236, thanks!

@victorjulien victorjulien deleted the smtp-frames/v22 branch June 8, 2024 07:30
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