Skip to content

Added support for RSASSA-PSS signature validation (#182)#183

Merged
CBonnell merged 6 commits intodigicert:mainfrom
Pearmint:main
Feb 26, 2026
Merged

Added support for RSASSA-PSS signature validation (#182)#183
CBonnell merged 6 commits intodigicert:mainfrom
Pearmint:main

Conversation

@Pearmint
Copy link
Copy Markdown
Contributor

@Pearmint Pearmint commented Feb 6, 2026

Added support for RSASSA-PSS signature validation (#182)

@CBonnell
Copy link
Copy Markdown
Collaborator

Hi @Pearmint, thanks for this PR! I added a few comments to specific lines, but can you also merge the "main" branch into your PR branch? It appears that there's been some changes to the Black code format recently that need to be incorporated for the CI pipeline to succeed.

@Pearmint
Copy link
Copy Markdown
Contributor Author

Hi @CBonnell,
I applied the suggested fixes and fixed that it would also work with different MGF salt lengths.
Are those changes fine?

@CBonnell
Copy link
Copy Markdown
Collaborator

Looks good but I think we need to future-proof how we obtain the salt length. I offered a proposal in an inline comment -- let know if you want to make that change, or if you prefer that I do that in a separate PR.

Copy link
Copy Markdown
Collaborator

@CBonnell CBonnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thank you for the contribution!

@CBonnell CBonnell merged commit d8f08be into digicert:main Feb 26, 2026
5 checks passed
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