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

smtp_forward: remove redundant outbound hook #2795

Closed
gramakri opened this issue May 6, 2020 · 0 comments · Fixed by #2796
Closed

smtp_forward: remove redundant outbound hook #2795

gramakri opened this issue May 6, 2020 · 0 comments · Fixed by #2796

Comments

@gramakri
Copy link
Collaborator

gramakri commented May 6, 2020

Expected behavior

When using smtp_forward for outbound emails, the mail gets delivered via the SMTP mail delivery queue (just like normal emails). I expect delivered hook to be called after mail is delivered, but it's not.

Observed behavior

Mail is delivered via the queue_outbound hook. Because, it ties into this hook, if a mail gets delivered, the delivered hook is never called.

Instead, by relying on the get_mx hook, the email flow is now the same for email when using smtp_forward or not.

gramakri added a commit to cloudron-io/Haraka that referenced this issue May 6, 2020
gramakri added a commit to cloudron-io/Haraka that referenced this issue May 6, 2020
gramakri added a commit to cloudron-io/Haraka that referenced this issue May 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant