Skip to content

Web crypto verify #1

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

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

Web crypto verify #1

wants to merge 10 commits into from

Conversation

RomanHotsiy
Copy link
Member

Needed for deno and web support.

* - Draft: see what is signed

* Fix bugs

* Fix bugs

* Revert "Fix bugs"

This reverts commit cd4b7d6fcb21cd1a40e57fd214c950c47c8667d5.

* Revert "Fix bugs"

This reverts commit 0947a89a04e16bedffc3cc5bf3152c6dedbe3a7d.

* Revert "- Draft: see what is signed"

This reverts commit 2a9c405f09f7843b994820f6933408d522a6d849.

* Patch for version 3

* Only parse references from canonicalized signed info

* Space

* Fixing spaces

* remove space

* Remove fix that does not apply

* Formatting

* Add warning about references.

* Remove formatting changes.

* Backport references, SignedInfo, and DigestValue patches.

* Only allow exclusive canon.

* Apply changes from v6 review

* Revert package-lock.json

* Apply suggested changes

---------

Co-authored-by: SecureSAML <[email protected]>
Co-authored-by: Blair Weber <[email protected]>
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