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

Python 3.12 and assorted fixes #232

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Python 3.12 and assorted fixes #232

wants to merge 5 commits into from

Conversation

bryango
Copy link

@bryango bryango commented May 17, 2022

This PR makes the app work for python 3.12. It incorporates the fix from #245 (comment). Closes:

Some other improvements:

Fix SSLV3_ALERT_HANDSHAKE_FAILURE for some imap servers

Connections to some imap servers (e.g. imap.pku.edu.cn) fail after upgrading to python 3.10. This is because they use old, less secure ciphers that are disabled for Python 3.10. In order to connect to those servers, we have to allow more ssl ciphers in imaplib2 for Python 3.10. See:

  1. https://stackoverflow.com/a/71007463
  2. https://www.openssl.org/docs/manmaster/man1/openssl-ciphers.html#CIPHER-LIST-FORMAT

Also,

  • Exit gracefully when autostart creation fails
  • Use raw strings for backlashes

Fix SSLV3_ALERT_HANDSHAKE_FAILURE for some imap servers.
They use old, less secure ciphers that are disabled for Python 3.10.
See <https://stackoverflow.com/a/71007463>.
@bryango bryango force-pushed the master branch 22 times, most recently from 159935a to 23b689f Compare May 24, 2024 15:38
bryango added 4 commits May 25, 2024 00:07
This fix was originally provided by Lalufu in #fedora-devel, and posted
on the issue tracker by @knurd.
This prevents SyntaxWarning: invalid escape sequence
@bryango bryango changed the title Fix SSLV3_ALERT_HANDSHAKE_FAILURE for some imap servers Python 3.12 and assorted fixes May 24, 2024
@tikank tikank mentioned this pull request Sep 29, 2024
9 tasks
@petterreinholdtsen
Copy link

Any hopes to get this and other fixes included in a new release?

@bryango
Copy link
Author

bryango commented Nov 6, 2024

Any hopes to get this and other fixes included in a new release?

It seems that mailnag is unmaintained. I will keep this PR open here, tracking my own minimal, personal fork. There are other forks with this patch merged while incorporating other improvements at the same time. You can track one of those forks (but with due caution! ⚠️ There is no guarantee for the security of forks (including my own) 😆

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.

2 participants