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

Wal 313 fix sd property check #839

Merged
merged 5 commits into from
Dec 10, 2024
Merged

Conversation

severinstampler
Copy link
Contributor

Description

Fixes problem with disclosed SD properties in presentations of SD-JWT credentials (ietf or W3C), not being verified against the signed hashes.
Disclosed properties are now checked, with the signature verification policy, and e2e tests trying to disclose forged properties have been implemented to ensure they are rejected.

Type of Change

  • bug fix - change which fixes an issue
  • new feature - change which adds functionality

Checklist

  • code cleanup and self-review
  • unit + e2e test coverage
  • documentation updated accordingly

Breaking

  • -

@severinstampler
Copy link
Contributor Author

Copy link

sonarqubecloud bot commented Dec 2, 2024

@taminobaumann taminobaumann requested review from philpotisk and removed request for chsavvaidis December 10, 2024 09:38
@philpotisk philpotisk merged commit 4de8aaf into main Dec 10, 2024
4 checks passed
@philpotisk philpotisk deleted the wal-313-fix-sd-property-check branch December 10, 2024 11:19
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