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

Feature/signature services #127

Draft
wants to merge 51 commits into
base: develop
Choose a base branch
from
Draft

Feature/signature services #127

wants to merge 51 commits into from

Conversation

n0900
Copy link
Contributor

@n0900 n0900 commented Sep 16, 2024

Adds remaining necessary data classes for the proposed UC5 remote signature flow.

This should finish the implementation of the OAuth2/CSC portion of the process.
Does not (yet) include OIDVCI extensions. Does not include any credential/info data classes or logic defined by CSC
Helper classes also still rough.

@n0900 n0900 requested a review from nodh September 16, 2024 08:18
Copy link
Contributor

@nodh nodh left a comment

Choose a reason for hiding this comment

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

Will look further if ready to review

@n0900
Copy link
Contributor Author

n0900 commented Sep 23, 2024

Rebase to #131

Copy link
Contributor

@nodh nodh left a comment

Choose a reason for hiding this comment

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

Please also add test code that uses the classes, if possible

@@ -79,6 +81,7 @@ sealed class AuthorizationDetails {
* Credential Configuration in the [IssuerMetadata.supportedCredentialConfigurations]. The Wallet MUST use these
* identifiers together with an Access Token in subsequent Credential Requests.
*/
// TODO is required in OID4VCI!
Copy link
Contributor

Choose a reason for hiding this comment

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

Yep, see #145

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For uc5 we get this after credential enrollment but this is out of scope for UC5 spec as well as CSC spec...

@@ -15,6 +15,7 @@ data class CredentialRequestParameters(
* specific parameters such as those defined in Appendix A MUST NOT be present
*/
@SerialName("credential_identifier")
// TODO Update
Copy link
Contributor

Choose a reason for hiding this comment

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

Todo?

import at.asitplus.wallet.lib.oauth2.OAuth2Client
import com.benasher44.uuid.uuid4

class RqesWalletService(
Copy link
Contributor

Choose a reason for hiding this comment

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

Unused class ... please use in tests

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