Skip to content

Smtp frames/v21#11223

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

Smtp frames/v21#11223
victorjulien wants to merge 12 commits intoOISF:masterfrom
victorjulien:smtp-frames/v21

Conversation

@victorjulien
Copy link
Member

'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 jasonish as a code owner June 3, 2024 17:51
@victorjulien victorjulien mentioned this pull request Jun 3, 2024
@codecov
Copy link

codecov bot commented Jun 3, 2024

Codecov Report

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

Project coverage is 83.03%. Comparing base (3b1fecb) to head (ec743f0).
Report is 23 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #11223    +/-   ##
========================================
  Coverage   83.02%   83.03%            
========================================
  Files         942      942            
  Lines      250606   250718   +112     
========================================
+ Hits       208067   208183   +116     
+ Misses      42539    42535     -4     
Flag Coverage Δ
fuzzcorpus 61.28% <69.44%> (-0.01%) ⬇️
livemode 18.69% <7.69%> (-0.01%) ⬇️
pcap 44.51% <67.13%> (-0.07%) ⬇️
suricata-verify 61.76% <97.20%> (+0.04%) ⬆️
unittests 60.62% <41.66%> (-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 20935

@victorjulien victorjulien added the needs rebase Needs rebase to main label Jun 4, 2024
@victorjulien victorjulien mentioned this pull request Jun 4, 2024
@victorjulien
Copy link
Member Author

replaced by #11229

@victorjulien victorjulien deleted the smtp-frames/v21 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

needs rebase Needs rebase to main

Development

Successfully merging this pull request may close these issues.

2 participants