-
Notifications
You must be signed in to change notification settings - Fork 15
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
fix: add validation to verification method and Issuer in VC #48
fix: add validation to verification method and Issuer in VC #48
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good :) Sorry for the late review
@borisrizov-zf Can we merge this? :) |
src/main/java/org/eclipse/tractusx/ssi/lib/proof/LinkedDataProofValidation.java
Outdated
Show resolved
Hide resolved
src/main/java/org/eclipse/tractusx/ssi/lib/proof/LinkedDataProofValidation.java
Show resolved
Hide resolved
@DominikPinsel gotta push out the Maven Central release first, otherwise we'll get a version mismatch between tractus-x 0.0.17 and the catenax-ng 0.0.17 |
@mustafasalfiti please rebase and we can merge. |
@mustafasalfiti I forgot, please add an entry in the |
3673fb1
to
935143d
Compare
You'll have to run |
b6cc373
to
ce1cdcb
Compare
ce1cdcb
to
ffdb0e2
Compare
we had a problem that a VC proof will be valid even if the first part of the verificationMethod inside the proof is not equal to the Issuer did and that should not be the case
in this PR we have added an extra layer of validation to check if the first part of the verificationMethod equal to the issuer of the VC
as well there is an extra test to check the correction of the code.