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-775: Verifier presentation definition policy bug fixes #861

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

cpatsonakis
Copy link
Contributor

Description

Provide a clear and concise description of the changes made in this pull request:

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

  • -

… required check for input descriptors, fix wallet credential matching algorithm to be based on jwt claims root payload, fix verifier default input descriptor constraints, fix e2e test cases
…ation definition policy testing from the verifier's perspective
… input descriptors are satisfied in the presentation definition policy evaluation, remove commented code, test cases for aforementioned changes, correct code indentation
…e wallet, refactor presentation definition policy code to main test package
@cpatsonakis cpatsonakis self-assigned this Dec 19, 2024
@cpatsonakis cpatsonakis added the bug Something isn't working label Dec 19, 2024
@walt-id walt-id locked and limited conversation to collaborators Dec 19, 2024
@cpatsonakis cpatsonakis marked this pull request as ready for review December 20, 2024 12:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant