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

Expand signature validation with the output of the signature together… #81

Closed
wants to merge 17 commits into from

Conversation

skriesch
Copy link
Collaborator

Philip has decoded the token and has added already the signature verification last week. But the output of the signature together with the signature validation was missing. That will be done with this Pull Request to close #11.

Signed-off-by: Sarah Julia Kriesch sarah.j.kriesch@fau.de

@skriesch skriesch requested a review from RumiAust June 28, 2022 13:21
… with header and payload #11

Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Copy link
Collaborator

@RumiAust RumiAust left a comment

Choose a reason for hiding this comment

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

The integration check is failing. Could you please have a look at that?

Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
Signed-off-by: Sarah Julia Kriesch <sarah.j.kriesch@fau.de>
@skriesch
Copy link
Collaborator Author

Anyway Jose says, that it does not know JWSObject, jose.JWS and other (should implemented inferfaces and utilities). Can you help me to fix this last issue in the line?

@skriesch skriesch closed this Jun 29, 2022
@skriesch skriesch deleted the 11-signature-validation branch June 29, 2022 12:06
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.

Validate Signature
2 participants