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

Add batch mail sending #1815

Open
wants to merge 7 commits into
base: development
from

Conversation

@Ilkrash
Copy link

Ilkrash commented Oct 24, 2019

#1813
This is the proposed solution to properly send a big number of mails at the same time, with opening and closing the connection with SMTP server only one time.

@Ilkrash Ilkrash requested a review from gobuffalo/core-managers as a code owner Oct 24, 2019
mail/internal/mail/smtp.go Outdated Show resolved Hide resolved
@Ilkrash

This comment has been minimized.

Copy link
Author

Ilkrash commented Oct 30, 2019

Please check the fix, tests on Windows failed due to "Test killed: ran too long (10m0s)"

@stanislas-m stanislas-m changed the base branch from master to development Nov 2, 2019
stanislas-m and others added 3 commits Nov 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.