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

Implement a destination that delivers to a local LMTP server #67

Merged
merged 3 commits into from
Feb 20, 2021

Conversation

Wuestengecko
Copy link
Contributor

LMTP is preferred over an LDA to deliver mail to Dovecot mailboxes. Additionally, LMTP functionality has been asked for in various placed across the web, and so far users who required it had to use fetchmail or other projects.

Note: This code is affected by bpo-42756 and will currently not work on an unpatched Python 3.9+ installation.

I've had this code running for a few weeks now and haven't noticed any issues — however, this getmail instance doesn't really see much traffic. If you spot anything, or want anything done differently, please feel free to comment!

If message delivery fails with a 5xx status code (i.e. permanently), the
message will instead be delivered to the specified fallback address.  If
delivery fails for that as well, the usual getmailDeliveryError is
raised.
After idling for too long, the LMTP server might terminate our
connection.  Instead of immediately raising an exception (and most
likely dying to it), try to reconnect to the server once first.
@emtiu
Copy link
Contributor

emtiu commented Feb 11, 2021

Thanks for your great effort! I've also got a getmail/dovecot-lda setup running, and I'll be sure to give this a spin (though it might be a few weeks before I find the time).

In any case, it might be prudent to wait before merging this until the latest mainline Python 3.x no longer fails with this code.

@Wuestengecko
Copy link
Contributor Author

In any case, it might be prudent to wait before merging this until the latest mainline Python 3.x no longer fails with this code.

That certainly sounds reasonable. It shouldn't be too long until the fixes land anyway, as they've already been merged last month and a new release is expected soon-ish.

@rpuntaie
Copy link
Collaborator

I merge this and will check if some if py_verion ... is needed regarding the mentioned bpo-42756 and its fix. Thanks.

@rpuntaie rpuntaie merged commit 834c466 into getmail6:master Feb 20, 2021
@rpuntaie rpuntaie mentioned this pull request Feb 27, 2021
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 this pull request may close these issues.

None yet

3 participants