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

Rsapss csr #890

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Rsapss csr #890

wants to merge 3 commits into from

Conversation

frk-pcn
Copy link

@frk-pcn frk-pcn commented Jun 29, 2021

Pull request for RSA PSS padding to handle

  • corrected calculation of salt length
  • supported scheme in case of CSR signing

@davidlehn
Copy link
Member

Do you have test cases for this change?
There are conflicts but it looks like it was rebased on main, not sure what's happening there.

@frk-pcn
Copy link
Author

frk-pcn commented Feb 24, 2022

I will see what's the best way to provide some tests to sign a CSR with RSA PSS padding. What do you think about changing tests/unit/csr.js to create a CSR in two flavors: with and without PSS padding?

@frk-pcn
Copy link
Author

frk-pcn commented Feb 25, 2022

I've enhanced the CSR test case. Unfortunately I'm not able to provide the change via github's pull request mechanism - currently no idea, why.

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.

None yet

2 participants