[MGNLMAIL-116] There should be a timeout on SMTP requests Created: 28/Sep/21 Updated: 01/Nov/21 Resolved: 18/Oct/21 |
|
| Status: | Closed |
| Project: | Magnolia Mail Module |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.5.9 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Maxime Michel | Assignee: | Michael Duerig |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | VN-Maintenance, artt | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | 1h | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Template: |
|
||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||
| Task DoD: |
[X]*
Doc/release notes changes? Comment present?
[X]*
Downstream builds green?
[X]*
Solution information and context easily available?
[X]*
Tests
[X]*
FixVersion filled and not yet released
[X] 
Architecture Decision Record (ADR)
|
||||||||||||
| Bug DoR: |
[X]*
Steps to reproduce, expected, and actual results filled
[X]*
Affected version filled
|
||||||||||||
| Documentation update required: |
Yes
|
||||||||||||
| Date of First Response: | |||||||||||||
| Description |
|
As we found out the hard way on the cloud, the default value for SMTP requests is to never time out. When the requests pile up, the mail server gets overloaded, leading Magnolia itself to crash. We now provide a default time out value of 5 seconds, which can be overriden via the timeoutInMillis field of SmtpConfiguration. |
| Comments |
| Comment by Michael Duerig [ 04/Oct/21 ] |
Analysis
ConclusionI propose to close this ticket until further information is available:
|
| Comment by David Lopez [ 05/Oct/21 ] |
|
Insights from chanh.hua
|