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

add s2s access token request for relying party #2568

Merged
merged 6 commits into from
Nov 10, 2023

Conversation

woutslakhorst
Copy link
Member

@woutslakhorst woutslakhorst commented Oct 27, 2023

closes #2504

This PR implements the access token request and uses the clientAPI to request an actual access token.

@woutslakhorst woutslakhorst marked this pull request as ready for review November 1, 2023 13:08
@woutslakhorst woutslakhorst force-pushed the feature/2504/vp_fo_s2s_flow branch from bd5ec4b to 7d63ee8 Compare November 1, 2023 13:11
@woutslakhorst woutslakhorst force-pushed the feature/2504/vp_fo_s2s_flow branch from 7d63ee8 to 1cba83e Compare November 1, 2023 13:13
auth/client/iam/client.go Outdated Show resolved Hide resolved
auth/client/iam/client.go Outdated Show resolved Hide resolved
auth/client/iam/client.go Show resolved Hide resolved
RequestRFC003AccessToken(ctx context.Context, jwtGrantToken string, authServerEndpoint url.URL) (*oauth.TokenResponse, error)

// RequestRFC021AccessToken is called by the local EHR node to request an access token from a remote Nuts node using Nuts RFC021.
RequestRFC021AccessToken(ctx context.Context, requestHolder did.DID, verifier did.DID, scopes string) (*oauth.TokenResponse, error)
Copy link
Member

Choose a reason for hiding this comment

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

not sure this is the right approach, given the variety of oauh2 and openID specs. But we need some moving around anyways, since the rest (OpenID4VP and VCI) is all over the place now

reinkrul

This comment was marked as off-topic.

auth/services/oauth/relying_party.go Outdated Show resolved Hide resolved
auth/services/oauth/relying_party.go Outdated Show resolved Hide resolved
auth/services/oauth/relying_party.go Outdated Show resolved Hide resolved
auth/services/oauth/relying_party.go Outdated Show resolved Hide resolved
auth/services/oauth/relying_party.go Outdated Show resolved Hide resolved
docs/_static/auth/iam.yaml Outdated Show resolved Hide resolved
vcr/vcr.go Show resolved Hide resolved
auth/services/oauth/relying_party.go Outdated Show resolved Hide resolved
@woutslakhorst woutslakhorst merged commit 06ea85a into master Nov 10, 2023
6 checks passed
@woutslakhorst woutslakhorst deleted the feature/2504/vp_fo_s2s_flow branch November 10, 2023 14:48
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.

construct VP in the correct format (check metadata)
2 participants