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

rsa signing selection, ECDH_RSA and ECDH_ECDSA #8

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

Conversation

Jafagervik
Copy link
Contributor

Testing some stuff for now, but I'm not sure the ECDH implementation need to differ too much in this section (might be wrong here, feel free to correct).

Also took a look at RSA signing selection, all tests passes but could create more tests anyways

@Jafagervik Jafagervik marked this pull request as ready for review October 29, 2024 10:06
@mookums
Copy link
Owner

mookums commented Oct 29, 2024

Hello! Thanks for the PR, I'll take a look at it soon!

@mookums
Copy link
Owner

mookums commented Nov 1, 2024

It looks good overall. Are you interested in adding in the keyx stuff and then TLS could be a lot further along?

@Jafagervik
Copy link
Contributor Author

Sorry for late answer! Been busy at work this week, but will try to continue on some more tasks next week if ok with you! Do you have any priority list regarding features or?

@Jafagervik
Copy link
Contributor Author

It looks good overall. Are you interested in adding in the keyx stuff and then TLS could be a lot further along?

Yes, can do that!

@mookums
Copy link
Owner

mookums commented Nov 8, 2024

No priority list at the moment, just the rest of the TODOs in the BearSSL file (which is mostly getting RSA keys to work as well). Appreciate your work so far!

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