-
Notifications
You must be signed in to change notification settings - Fork 0
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
Configurable SMTP Timeout (GSI-986) #20
Conversation
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.
Looks good, just one suggestion
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.
Why the default is None? I would expect, as a default behavior, an error message eventually if there is a problem in connection rather than infinite waiting. Then adjusting the value accordingly if necessary. So I would say 60 seconds.
Pull Request Test Coverage Report for Build 10737100864Details
💛 - Coveralls |
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.
Looks good to me now, thanks.
Adds an optional
smtp_timeout
config option. By default it willmaintain the prior infini-hang behaviourtime out after 60 seconds. Added an error class to theSMTPClient
to catch any instance ofOSError
raised by a rejected connection or timeout.Added in some debug statements in the SMTP module to aid in future bug hunting.
Bumped the version from
2.0.0
to2.1.0
.