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.

Sign up for GitHub

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

Jump to bottom

chore: fix after IATP changes #4012

Merged
merged 1 commit into from
Mar 18, 2024
Merged

chore: fix after IATP changes #4012

merged 1 commit into from
Mar 18, 2024

Conversation

wolf4ood
Copy link
Contributor

@wolf4ood wolf4ood commented Mar 15, 2024 •
edited
Loading

What this PR changes/adds

Fixes after IATP changes about renaming access_token to token.

Also in this PR:

  • Fixes some case of Verifiable Presentation verification in JWT format
  • Removed the nbf claim verification
  • Changed the API path for the presentation query to {base_path}/presentations/query

@jimmarino jimmarino added the bug Something isn't working label Mar 15, 2024
@wolf4ood wolf4ood marked this pull request as ready for review March 18, 2024 06:57
@wolf4ood wolf4ood requested a review from ndr-brt as a code owner March 18, 2024 06:57
@wolf4ood wolf4ood merged commit 4b3ec48 into eclipse-edc:main Mar 18, 2024
22 of 23 checks passed
@wolf4ood wolf4ood deleted the chore/fix_with_latest_iatp_changes branch March 18, 2024 08:11
@wolf4ood wolf4ood mentioned this pull request Mar 18, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jimmarino jimmarino jimmarino approved these changes

@ndr-brt ndr-brt ndr-brt approved these changes

@paullatzelsperger paullatzelsperger Awaiting requested review from paullatzelsperger

Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@wolf4ood @jimmarino @ndr-brt