Skip to content
This repository has been archived by the owner on Oct 6, 2020. It is now read-only.

Does not disconnect/reconnect, premature connect. #9

Open
azilber opened this issue Jun 27, 2019 · 0 comments
Open

Does not disconnect/reconnect, premature connect. #9

azilber opened this issue Jun 27, 2019 · 0 comments

Comments

@azilber
Copy link

azilber commented Jun 27, 2019

Having a few issues with this plugin:

  1. SMTP is immediately established upon log.Hooks.Add
    This causes a timeout and a Failed to fire hook: write tcp 10.1.1.187:59875->167.114.101.158:25: write: broken pipe error when a log.Alert/Fatal is eventually fired because the server won't hold an SMTP connection forever. Timeouts are usually around 30s to prevent DDOS attacks.
  2. If an alert is generated before the SMTP server timeout, then the SMTP connection is not disconnected with a "QUIT". Instead of "QUIT" further "DATA" commands are issued which cause errors.
  3. Because there is never a proper termination/reconnection issue 1 or 2 are fatal and logrus_mail never recovers.

Here's a wireshark capture using the public mail.sharklasers.com email host for testing:
Screenshot 2019-06-27 at 5 12 48 PM

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant