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.

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

memory limit? #105

Open
porjo opened this issue Aug 4, 2016 · 3 comments · May be fixed by mailhog/storage#3
Open

memory limit? #105

porjo opened this issue Aug 4, 2016 · 3 comments · May be fixed by mailhog/storage#3

Comments

@porjo
Copy link

porjo commented Aug 4, 2016

MailHog doesn't appear to impose an upper limit on the amount of memory it could allocate to the message queue. Assuming that people aren't going to bother deleting email most of the time, won't this become a time bomb on a long-running server?

A sensible default limit should be imposed. Question is what to do once that limit is reached - do you auto-delete old messages or return a non-delivery error?

Relates to #74

@ian-kent
Copy link
Member

Yeah, this definitely needs fixing!

I don't think the two PRs linked above really solve this issue, I'll add something specifically to limit memory consumption

@meranos
Copy link

meranos commented Sep 6, 2017

Any news on this one? We would love to use MailHog in our testing environment, but we are a bit concerned about long term memory usage.

@allangood
Copy link

Seems to be duplicated from #74
I've sent the PR #244 to address this!

Regards.

@tyndyll tyndyll added this to the v1.1 milestone Aug 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants