Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix #15: correctly parse esmtp args on MAIL and RCPT #16

Merged
merged 2 commits into from
Oct 5, 2023
Merged

fix #15: correctly parse esmtp args on MAIL and RCPT #16

merged 2 commits into from
Oct 5, 2023

Conversation

hxdmp
Copy link

@hxdmp hxdmp commented Oct 5, 2023

No description provided.

@hxdmp hxdmp mentioned this pull request Oct 5, 2023
Closed
@coveralls
Copy link
Collaborator

coveralls commented Oct 5, 2023 •
edited
Loading

Coverage Status

coverage: 85.796% (+0.03%) from 85.77% when pulling d824e79 on hxdmp:main into 30d4d62 on d--j:main.

@d--j d--j linked an issue Oct 5, 2023 that may be closed by this pull request
ESMTP args truncated in RCPT TO stage #15
Closed
@d--j d--j merged commit 6cc737f into d--j:main Oct 5, 2023
4 checks passed
@d--j
Copy link
Owner

d--j commented Oct 5, 2023

Thanks for the PR, I just added tests for this new functionality

@hxdmp
Copy link
Author

hxdmp commented Oct 6, 2023

Thank you for your quick reaction and help - and also for taking the extra mile to add tests to this PR. I really appreciate that! 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

ESMTP args truncated in RCPT TO stage
3 participants
@hxdmp @coveralls @d--j