Skip to content

SMTP: improve parsing of multiline headers#160

Merged
roxblnfk merged 1 commit intomasterfrom
smtp-headers
Jan 23, 2025
Merged

SMTP: improve parsing of multiline headers#160
roxblnfk merged 1 commit intomasterfrom
smtp-headers

Conversation

@roxblnfk
Copy link
Copy Markdown
Member

What was changed

Supported multiline headers parsing

Why?

See RFC822 section 3.1.3)

Checklist

@github-actions github-actions Bot added the tests Changes in the tests label Jan 23, 2025
@roxblnfk roxblnfk force-pushed the smtp-headers branch 2 times, most recently from a058680 to 020cb2a Compare January 23, 2025 20:23
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 23, 2025

Codecov Report

Attention: Patch coverage is 95.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 23.60%. Comparing base (4359f6b) to head (9afcdac).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
src/Traffic/Parser/Smtp.php 95.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #160      +/-   ##
============================================
+ Coverage     23.27%   23.60%   +0.33%     
- Complexity     1343     1355      +12     
============================================
  Files           159      159              
  Lines          4348     4367      +19     
============================================
+ Hits           1012     1031      +19     
  Misses         3336     3336              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@roxblnfk roxblnfk merged commit 001c625 into master Jan 23, 2025
@roxblnfk roxblnfk deleted the smtp-headers branch January 23, 2025 20:30
@roxblnfk roxblnfk changed the title SMTP: improve parsing of multiline headers ( SMTP: improve parsing of multiline headers Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Changes in the tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Long Subject with spaces is trimmed

1 participant