Skip to content

Conversation

@acoburn
Copy link
Collaborator

@acoburn acoburn commented May 5, 2023

At present, the access grant library uses the default serialization of java.time.Instant, which results in a long, which is not supported by a VC-API endpoint. Instead, the value needs to be serialized as an ISO-8601 string.

In addition, the revocation method uses the status list identifier rather than the access grant identifier, which also causes an error.

@acoburn acoburn requested a review from a team as a code owner May 5, 2023 18:12
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 18:12 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 18:12 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS DevNext May 5, 2023 18:12 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 19:54 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 19:54 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS DevNext May 5, 2023 19:54 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 20:10 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 20:10 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS DevNext May 5, 2023 20:10 — with GitHub Actions Inactive
@acoburn acoburn force-pushed the JCL-337-access-grant-issue-serialization branch from ef0b0f5 to 55edf7e Compare May 5, 2023 20:21
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 20:21 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS PodSpaces May 5, 2023 20:21 — with GitHub Actions Inactive
@acoburn acoburn temporarily deployed to ESS DevNext May 5, 2023 20:21 — with GitHub Actions Inactive
@timea-solid timea-solid temporarily deployed to ESS PodSpaces May 8, 2023 09:31 — with GitHub Actions Inactive
@timea-solid timea-solid temporarily deployed to ESS PodSpaces May 8, 2023 09:31 — with GitHub Actions Inactive
@timea-solid timea-solid temporarily deployed to ESS DevNext May 8, 2023 09:31 — with GitHub Actions Inactive
Copy link
Contributor

@timea-solid timea-solid left a comment

Choose a reason for hiding this comment

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

I confirm this is working.
Fixed the failing e2e tests too.

@timea-solid timea-solid merged commit 9aac8b9 into main May 8, 2023
@timea-solid timea-solid deleted the JCL-337-access-grant-issue-serialization branch May 8, 2023 10:08
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.

3 participants