Skip to content

[draft] IMAP protocol parser and logger#14722

Draft
glongo wants to merge 2 commits intoOISF:mainfrom
glongo:dev-imap-proto-v1
Draft

[draft] IMAP protocol parser and logger#14722
glongo wants to merge 2 commits intoOISF:mainfrom
glongo:dev-imap-proto-v1

Conversation

@glongo
Copy link
Contributor

@glongo glongo commented Jan 30, 2026

IMAP commands, email headers, and body are parsed; the JSON schema still needs to be defined.
After that, I can proceed with a PR.

@glongo glongo marked this pull request as draft January 30, 2026 07:44
@github-actions
Copy link

NOTE: This PR may contain new authors.

@codecov
Copy link

codecov bot commented Jan 30, 2026

Codecov Report

❌ Patch coverage is 59.59210% with 634 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.01%. Comparing base (2cf9a32) to head (5f9bcbd).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #14722      +/-   ##
==========================================
- Coverage   82.14%   82.01%   -0.14%     
==========================================
  Files        1007     1009       +2     
  Lines      263194   264729    +1535     
==========================================
+ Hits       216210   217118     +908     
- Misses      46984    47611     +627     
Flag Coverage Δ
fuzzcorpus 59.82% <4.47%> (-0.38%) ⬇️
livemode 18.66% <4.47%> (-0.22%) ⬇️
pcap 44.37% <9.04%> (-0.23%) ⬇️
suricata-verify 64.97% <9.23%> (-0.39%) ⬇️
unittests 59.35% <59.11%> (-0.01%) ⬇️

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@suricata-qa
Copy link

Information: QA ran without warnings.

Pipeline = 29347

@catenacyber
Copy link
Contributor

Looks cool.

What about keywords ?

@glongo
Copy link
Contributor Author

glongo commented Feb 2, 2026

Looks cool.

What about keywords ?

I'll implement them after the JSON schema is defined.

@catenacyber
Copy link
Contributor

Ok, do you have some SV tests where we can see the logging ?

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.

3 participants