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

feat: add Credential Request endpoints (IdentityApi) #590

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

Conversation

paullatzelsperger
Copy link
Member

@paullatzelsperger paullatzelsperger commented Feb 14, 2025

What this PR changes/adds

this PR adds two endpoints to the IdentityApi, specifically the VerifiableCredentials API to initiate credential requests, and to obtain
credential requests.

Why it does that

to be able to initiate credential requests and to get them later

Further notes

  • a previous renaming was reverted (IssuerCredentialIssuanceProcess -> IssuanceProcess), because on the holder side we'll call it "CredentialRequest", on the issuer side we'll call it "IssuanceProcess"

Who will sponsor this feature?

Please @-mention the committer that will sponsor your feature.

Linked Issue(s)

Closes #586

Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

@paullatzelsperger paullatzelsperger force-pushed the add_credentialrequest_endpoints branch from 8f5c79a to 88bd15b Compare February 14, 2025 14:16
content = @Content(array = @ArraySchema(schema = @Schema(implementation = ApiErrorDetail.class)), mediaType = "application/json"))
}
)
HolderCredentialRequestDto getCredentialRequest(String participantContextId, String issuanceProcessId, SecurityContext securityContext);

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'issuanceProcessId' is never used.
@@ -126,8 +126,7 @@
return this;
}

@Override
public IssuerCredentialIssuanceProcess build() {
public IssuanceProcess build() {

Check notice

Code scanning / CodeQL

Missing Override annotation Note

This method overrides
Builder<IssuanceProcess,Builder>.build
; it is advisable to add an Override annotation.
@@ -124,6 +126,11 @@
return revocationService;
}

@Provider(isDefault = true)
public CredentialRequestService createDefaultCredentialRequestService(ServiceExtensionContext context) {

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'context' is never used.
* @param typesAndFormats A map containing credential-type - credential-format entries
* @return A ServiceResult containing the issuer-assigned process ID, or a failure otherwise.
*/
ServiceResult<String> initiateRequest(String issuerDid, String requestId, Map<String, String> typesAndFormats);

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'issuerDid' is never used.
* @param typesAndFormats A map containing credential-type - credential-format entries
* @return A ServiceResult containing the issuer-assigned process ID, or a failure otherwise.
*/
ServiceResult<String> initiateRequest(String issuerDid, String requestId, Map<String, String> typesAndFormats);

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'requestId' is never used.
* @param typesAndFormats A map containing credential-type - credential-format entries
* @return A ServiceResult containing the issuer-assigned process ID, or a failure otherwise.
*/
ServiceResult<String> initiateRequest(String issuerDid, String requestId, Map<String, String> typesAndFormats);

Check notice

Code scanning / CodeQL

Useless parameter Note

The parameter 'typesAndFormats' is never used.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Credential request: add IdentityAPI endpoints for credential requests
3 participants