-
-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix connection issues #51
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #51 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 9 10 +1
Lines 355 360 +5
=========================================
+ Hits 355 360 +5
☔ View full report in Codecov by Sentry. |
1362620
to
49e2a02
Compare
Render markdown emails before opening a SMTP connection to avoid connection timeouts. Services like AWS SES have very short connection timeouts. Therefore, all messages should be fully rendered before a connection is established.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, I feel better with this change. :)
Just left a couple of small comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, lets gooooo
Render markdown emails before opening a SMTP connection to avoid
connection timeouts. Services like AWS SES have very short
connection timeouts. Therefore, all messages should be fully
rendered before a connection is established.