Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Oct 25, 2024. It is now read-only.

hs2019 with PSS support#22

Open
fulder wants to merge 33 commits intoahknight:masterahknight/httpsig:masterfrom
fulder:hs2019-supportfulder/python-httpsig:hs2019-supportCopy head branch name to clipboard
Open

hs2019 with PSS support#22
fulder wants to merge 33 commits intoahknight:masterahknight/httpsig:masterfrom
fulder:hs2019-supportfulder/python-httpsig:hs2019-supportCopy head branch name to clipboard

Conversation

@fulder
Copy link

@fulder fulder commented Aug 25, 2020

Implement support for new hs2019 algorithm together with PSS digital signature algorithm in order to follow the latest v12 draft

As the digital signature algorithm should now be derived from the keyId both the Signer and Verifier classes are now accepting a new (optional) sign_algorithm parameter which should be a subclass of the new SignAlgorithm superclass. This way we can easy add support for other digital signature algorithms by adding a custom class (implementing the sign and verify functions).

This has been implemented in parallel with the previously accepted algorithms (e.g. rsa-sha256). This means these algorithms are still supported although a deprecation message is now printed while they are used and they should probably be removed in next major release of this library (together with some cleanup of the 'old' code).

The test_signature tests has been updated to the new hs2019 with one test left for the deprecated rsa-sha256. The README file has also been updated with the new algorithm.

fulder added 25 commits August 24, 2020 11:06
In order to be able to decode the PSS message, the salt length need to
be known.
@McLoone
Copy link

McLoone commented Aug 27, 2020

+1

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.