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

use keepalive-socket for cross-platform support #7

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

tvogel
Copy link

@tvogel tvogel commented Aug 28, 2024

On Linux, I received a

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/home/til/git/MailboxCleanup/src/mailbox_cli.py", line 133, in <module>
    main()
  File "/home/til/git/MailboxCleanup/src/mailbox_cli.py", line 102, in main
    imap.login()
  File "/home/til/git/MailboxCleanup/src/mailbox_imap.py", line 84, in login
    self.imap.sock.setsockopt(socket.IPPROTO_TCP, TCP_KEEPALIVE, 3)
OSError: [Errno 22] Invalid argument

So, I looked for a cross-platform solution, found https://stackoverflow.com/a/14855726/119725 and https://stackoverflow.com/a/76668530/119725 and decided to use the latter.

I hope, you find it useful. Thanks for the great project!

PS. Sorry for the PR noise - I used main as the source branch before and that is not good...

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.

1 participant