Skip to content

add getters for rsa key params p, q, n, e and d#4

Open
repomaa wants to merge 1 commit into
randomstate:masterfrom
repomaa:topic/key_param_getters
Open

add getters for rsa key params p, q, n, e and d#4
repomaa wants to merge 1 commit into
randomstate:masterfrom
repomaa:topic/key_param_getters

Conversation

@repomaa

@repomaa repomaa commented Apr 8, 2019

Copy link
Copy Markdown

This should be useful for serializing into JWK

@byfoz

byfoz commented Apr 29, 2019

Copy link
Copy Markdown
Member

Thanks @jreinert . We are trying to get this integrated into the Crystal stdlib so that's the reason I haven't merged this in yet (if at all). @bararchy was preparing the PR for the stdlib. @bararchy is there any update on this? Wondering how we can merge this in with the stdlib in mind 💪

@stakach

stakach commented Jun 14, 2019

Copy link
Copy Markdown

I think just merge this in. Might as well keep this project running in the mean time.
Also I'm happy to take over maintenance if you want to make me a collaborator (if you don't mind)

You can see some of my companies crystal work over here: https://github.com/spider-gazelle

Side note, this pull doesn't work properly with LibreSSL

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.

3 participants