-
Notifications
You must be signed in to change notification settings - Fork 242
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
What's needed for 3.1.0 release? #145
Comments
Cant wait to get rid of the deprecation warning (#124) 👍 |
Sorry I've been so MIA. I am writing this late at night after a long work week The last thing I would like to see merged for 3.1.0 is my PR #100. Unfortunately, @zejn and I noted that the PR needs to support RFC8410. When I attempted to add PEM deserialization support, it looked a lot like this: I looked into using the pyasn1 package to deserialize Ed25519 keys in RFC8410 format, since we're already depending on that library elsewhere, but it didn't have them built-in. So then I looked into what it would take to add them, and the pyasn1 project directed me to add a compiled module to the pyasn1-modules repository:
So then I looked into adding a data structure for RFC8410 to pyasn1-modules:
So then I looked into using the ans1ate tool to transform the actual RFC8410 definitions into Python code. I created an So then I looked into the imports used by RFC8410, to see if I could get them asn1ated into Python. I figured I could probably concatenate them all together and feed them into asn1ate. But digging through the labyrinth of RFC references I quickly got lost and confused, which then became demoralized and intimidated by the task at hand. I'm just a developer who's trying to make crypto software easier to use, I am not an expert on RFCs and the various standard bodies. As I was retracing my steps and writing this all out, I did stumble onto the pycrate project, which does seem to have all of the necessary ASN definitions: Maybe those can be used to generate a Python definition for RFC8410? @mattsb42-aws If you would like to help out, please get RFC8410 supported by pyasn-modules. If you are feeling particularly productive, please open an PR against my personal ed25519 branch that adds serialization/deserialization support. Everything I merge into that branch will become part of this repository's PR #100. Since I did a lot of work on that and I am a bit vain, I would like to get at least partial credit for getting Ed25519 support working. I'll try to work on this soon, but I can't promise anything. If you can get RFC8410 support working, I'll get the tests for PR #100 passing again. Once that's merged I will release version 3.1.0 on PyPI and GitHub. [1] Copied and pasted here for your convenience:
|
Thanks @blag, I'll take a look. |
I'm going to try to avoid this if at all possible. LGPL is...problematic... |
Resolved by #157. That's where release prep is being done. |
I don't have permissions to upload to PyPI, but I have emailed @mpdavis to add me as a maintainer there. Once that is done, I will upload it. Until then, I have made a release on GitHub. |
Thanks for driving this through, @blag :) |
Ladies and gentlemen, version 3.1.0. |
Shoutout to @blag for being proactive in getting this out. 👏 |
Thanks, @blag! |
What's needed for the 3.1.0 release to happen? Is there anything I can help with?
The text was updated successfully, but these errors were encountered: