Skip to content

Update parser.rs by issue #1464 #1465

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
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update parser.rs by issue #1464 #1465

wants to merge 1 commit into from

Conversation

UstyuzhaninAV
Copy link

added new pattern form full mailboxes at gmx.de

@UstyuzhaninAV
Update parser.rs
7f36ab5 
added new pattern form full mailboxes at gmx.de
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@UstyuzhaninAV UstyuzhaninAV mentioned this pull request Apr 30, 2024
Open
darvell
darvell reviewed May 9, 2024
View reviewed changes
core/src/smtp/parser.rs
// 550 user has too many messages on the server
|| e.contains("too many messages")
// https://newsroom.gmx.net/2017/09/28/unzustellbar-fehlermeldungen-email/
|| e.comtains("exceeded storage allocation")
Copy link

Choose a reason for hiding this comment

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

contains, not comtains.

Copy link
Member

Choose a reason for hiding this comment

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

any updates on this? I'll merge once fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@amaury1093 amaury1093 amaury1093 left review comments

+1 more reviewer

@darvell darvell darvell left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@UstyuzhaninAV @CLAassistant @darvell @amaury1093