-
Notifications
You must be signed in to change notification settings - Fork 12
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
feature/SDK-10 #184
base: develop
Are you sure you want to change the base?
feature/SDK-10 #184
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.
This will need a significant amount of changes. Never ever require private keys in methods, except for the KMS no agent plugin can access private keys
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/types/IEBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/types/IEBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/types/IEBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
* @property {number} [exp] Expires at | ||
* @property {string} [iss] Issuer (Authorisation API) | ||
*/ | ||
export interface Ake1SigPayload { |
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.
What is this doing here?
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.
This is the response from the EBSI RP, at least part of it
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.
Not needed anymore
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
packages/ebsi-authorization-client/src/agent/EBSIAuthorizationClient.ts
Outdated
Show resolved
Hide resolved
grant_type: 'vp_token', | ||
vp_token: vpJwt, | ||
presentation_submission: presentationSubmission, | ||
scope: EBSIScope.didr_invite, |
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.
Should this scope
come from args instead of being hardcoded?
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.
Retrieving the values from args now
…ndpoints, added oid4vci,
No description provided.