FEATURE: Add IMAP IDLE support for getmail#4675
Conversation
β¦lback location.
|
Documentation preview for this PR is ready! π Built with commit: b9477ef |
|
Thank you for your contribution. Just wanted to let you know, that we are quiet busy and review will take same time. |
|
I will also take a look, but all PRs are currently blocked for me because of #4536 |
|
Oh I thought this was scheduled for v16 π I've had a glance over the PR diff but I would like to try take a simpler approach which I've outlined here: #4713 (comment) That's dependent upon upstream and DMS switching back to using the PyPi package instead of the Debian one to benefit from that. Taking the approach I've outlined would roughly reduce down to this: [program:getmail-parallel]
numprocs = 2
numprocs_start = 1
process_name = %(program_name)s-%(process_num)s
autorestart = true
stopasgroup = true
command = bash -c 'getmail --getmaildir /var/lib/getmail --rcfile getmailrc-account-%(process_num)s && sleep %(ENV_GETMAIL_POLL)sm'NOTE: Until such a PR is feasible, any DMS user could add the above as a config mount into For the IDLE support, add the Or adapt the approach you've got here. It's just more of a hassle to merge a feature when it could be greatly simplified by getting support for the idle setting in config files upstream π |
|
I spent more time on investigating this integration with upstream.
The latest unreleased Getmail commits have added the ability to use Arguably the feature itself as I have described it (and as it is vaguely documented at the Getmail6 project) is in itself broken and not working as implied/documented. Furthermore, I'd like to know the benefit of enabling/supporting IDLE, given the single mailbox limitation?
I assume it's very few remote accounts and quite possibly only the default INBOX being monitored. I'm just curious how important the feature actually is to you, or if it's more of just an ick with polling often instead of receiving a notification from the remote server? Related history for Fetchmail idle support:
|
|
Personally I don't consider the single folder a limitation. The issue I find with polling is that you need to get the interval down to a minute maximum. |
Fair enough, thank you :)
I've yet to get around to filing a bug report properly upstream, but I have pointed it out to the maintainer already that the It is designed to specify a mailbox to receive a notification about updates from, and yet there's no documentation to note that configuring that alone is insufficient if your mailbox is not As this PR lacks a test case for catching that, users are just as likely to make that mistake unaware of it (I'm kind of surprised that it hasn't been raised as a bug report upstream thus far, but perhaps nobody is bothering to use Getmail with mailboxes other than INBOX π€·ββοΈ) I'll return to this PR after I've approved the Dovecot 2.4 / Debian 13 upgrade PR. I'd much prefer convincing upstream to improve so integration in DMS has reduced complexity. Apologies for the delay that causes here, but thus far I think you're the only user that's requested support for the feature in DMS π I don't mind supporting integrations that can justify the convenience for users well enough, but I also would very much like it done right so that it's minimal burden to maintainers to carry. |
Similar to the FETCHMAIL_PARALLEL option which enables the use of IMAP IDLE, this PR provides the same functionality for getmail.
It builds upon the existing code as it doesn't touch the config file generation. Instead it only focusses on the start of getmail.
This should provide the functionality requested in discussion #4667
Type of change
Checklist
docs/)CHANGELOG.md