-
Notifications
You must be signed in to change notification settings - Fork 34
Refactor/ssl context #28
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
base: master
Are you sure you want to change the base?
Conversation
|
LGTM. With this, tuya-convert works on Python 3.13. Without it, it does not. |
|
I don't think this package is going to be maintained and also python 3.13 has this feature builtin so it's no longer necessary. If anyone is interested, I forked this repo into my own sslpsk-pmd3 quite a long time ago |
|
It's not strictly necessary, but tuya-convert is written to use this, and with sslpsk patched with this PR, tuya-convert works as-is. To use the new Python built-in PSK support, tuya-convert would need a patch. Also, I'll send a PR for tuya-convert to use your fork for now, I guess. Not sure if I want to figure out how to rejig tuya-convert to work with the built-in PSK stuff. Thanks. |
This is the stuff I found I needed to make this script work on Fedora. We also install the branch of sslpsk that backs this PR: drbild/sslpsk#28 because current upstream sslpsk does not work on Python 3.12+, and so does not work on current stable Fedora. Signed-off-by: Adam Williamson <awilliam@redhat.com>
This is the stuff I found I needed to make this script work on Fedora. We also install the branch of sslpsk that backs this PR: drbild/sslpsk#28 because current upstream sslpsk does not work on Python 3.12+, and so does not work on current stable Fedora. Signed-off-by: Adam Williamson <awilliam@redhat.com>
|
There is an updated version called sslpsk3 v2.0.0, which I just released with support for |
No description provided.