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

Large messages are not accepted correctly #389

Closed
foxcpp opened this issue Aug 22, 2021 Discussed in #388 · 3 comments
Closed

Large messages are not accepted correctly #389

foxcpp opened this issue Aug 22, 2021 Discussed in #388 · 3 comments
Assignees
@foxcpp
Labels
bug Something isn't working. mta-in Related to incoming message processing part of the MTA functionality (mail exchanger).

Comments

@foxcpp
Copy link
Owner

foxcpp commented Aug 22, 2021

Discussed in #388

Originally posted by jpatters August 22, 2021
I am having issues with attachments. The largest attachment I've gotten to go though successfully is 598KB. What size attachment should I be able to send/receive with the default configuration? I expected it would be 32MB based on the default value of max_message_size.
I am using S3 for message storage but I have also tried with fs storage and had the same result.

Thanks in advance.
@foxcpp foxcpp added bug Something isn't working. mta-in Related to incoming message processing part of the MTA functionality (mail exchanger). labels Aug 22, 2021
@foxcpp foxcpp self-assigned this Aug 22, 2021
@foxcpp
Copy link
Owner Author

foxcpp commented Aug 22, 2021 •
edited
Loading

I sent a 1.2 MiB message to maddy process and it got stuck in a loop with 100% CPU use.

@foxcpp
Copy link
Owner Author

foxcpp commented Aug 22, 2021

Seems to be the regression introduced in 426c618, affecting 0.5.0.

@foxcpp foxcpp closed this as completed in 820d918 Aug 22, 2021
@foxcpp
Copy link
Owner Author

foxcpp commented Aug 22, 2021 •
edited
Loading

Workaround for 0.5.0 is to set max_header_size to something larger than max_message_size:

smtp ... {
  ...
  max_header_size 33M
}

@foxcpp foxcpp mentioned this issue Sep 22, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@foxcpp foxcpp

Labels
bug Something isn't working. mta-in Related to incoming message processing part of the MTA functionality (mail exchanger).
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@foxcpp