mirror of
https://github.com/axllent/mailpit.git
synced 2026-04-26 08:45:54 +03:00
[GH-ISSUE #407] 501 5.5.4 Syntax error in parameters or arguments (invalid SIZE parameter) #264
Labels
No labels
awaiting feedback
bug
docker
documentation
enhancement
github_actions
invalid
pull-request
question
stale
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/mailpit#264
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @baiomys on GitHub (Dec 12, 2024).
Original GitHub issue: https://github.com/axllent/mailpit/issues/407
Something nasty happening while proxying email via Cloudflare.
Two different senders, same result:
@axllent commented on GitHub (Dec 12, 2024):
I think this may be related to to some (currently) unsupported functionality in the smtpd server in Mailpit (I am investigating, however I just had surgery so I'm on very strong pain medication and my brain isn't functioning properly :)).
I'm fairly sure it has to do with the secondary command in
MAIL FROM:<SRS0=I7TD=ft=sender.ozon.com=mailer@2tg.org> BODY=8BITMIME(ie:BODY=8BITMIME) which is causing the error in Mailpit. I need to do a lot of reading before I can comment further.In the meantime, can you maybe answer a question: do you get this error if you send directly to Mailpit? I'm curious as to whether it is the CloudFlare proxy that is adding that, or your SMTP client.
@baiomys commented on GitHub (Dec 12, 2024):
This weird behavior caused by Cloudflare forwarding in one mailbox to one mailbox rule.
Catch-all rule (many to one) seems working fine and sending mail directly to mailpit works fine.
It does not depend on MTA because I tested two completely different senders.
@axllent commented on GitHub (Dec 13, 2024):
I believe in some cases SMTP relay servers/proxies (in this case CloudFlare) may be adding the
BODY=...parameter to assist with message encoding with downstream bounces. Mailpit was originally detecting any parameter as a "size value" which was breaking this. I've pushed a fix for this which is currently in the Docker edge build (unreleased) in case you wanted to test early (if you are able to test please).Hopefully I will have a new release out by the end of the weekend.
@baiomys commented on GitHub (Dec 13, 2024):
There is some progress evident.
=)
REAL message originated NOT from cloudflare was not delivered:
TEST message from sendtestemail WAS delivered:
It is obvious that:
Maybe you should consider splitting MAIL FROM string on "> " or " " delimiter, trimming all spaces and processing list as separate commands?
@axllent commented on GitHub (Dec 13, 2024):
Damn, that was my mistake, sorry! I have just pushed a fix for the fix. If you don't mind, could you please test again with the latest edge build and let me know if this now resolves the issue for you? Thank you!
@baiomys commented on GitHub (Dec 13, 2024):
You fixed it!
Fantastic, taking into account your current condition.
I am unable to check status of emails containing SIZE in MAIL FROM string, but I hope it will be also okay.
Thanks for quick fix.
@axllent commented on GitHub (Dec 13, 2024):
Yes, I'm glad to hear it! I'm typing everything with one hand which is really slow - but I have time :) The size argument shouldn't be a problem 👍
@axllent commented on GitHub (Dec 14, 2024):
This has now been released in v1.21.7.