refactor: suggestion to inject selectedCredentials in initializeAutho…

…rization of wallet provider
walt-id · Nov 22, 2023 · a3bb7b1 · a3bb7b1
1 parent a597db9
commit a3bb7b1
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/main/kotlin/id/walt/service/SSIKit2WalletService.kt b/src/main/kotlin/id/walt/service/SSIKit2WalletService.kt
@@ -189,8 +189,7 @@ class SSIKit2WalletService(accountId: UUID, walletId: UUID) : WalletService(acco
 
         println("USING PRESENTATION REQUEST, SELECTED CREDENTIALS: $selectedCredentialIds")
 
-        val presentationSession = credentialWallet.initializeAuthorization(authReq, 60.seconds)
-            .copy(selectedCredentialIds = selectedCredentialIds.toSet())
+        val presentationSession = credentialWallet.initializeAuthorization(authReq, 60.seconds, selectedCredentialIds.toSet())
 
         println("Resolved presentation definition: ${presentationSession.authorizationRequest!!.presentationDefinition!!.toJSONString()}")
 

diff --git a/src/main/kotlin/id/walt/service/oidc4vc/TestCredentialWallet.kt b/src/main/kotlin/id/walt/service/oidc4vc/TestCredentialWallet.kt
@@ -32,6 +32,7 @@ import kotlinx.uuid.UUID
 import kotlinx.uuid.generateUUID
 import kotlin.io.encoding.Base64
 import kotlin.io.encoding.ExperimentalEncodingApi
+import kotlin.time.Duration
 import kotlin.time.Duration.Companion.minutes
 
 const val WALLET_PORT = 8001
@@ -245,4 +246,14 @@ class TestCredentialWallet(
     fun parsePresentationRequest(request: String): AuthorizationRequest {
         return resolveVPAuthorizationParameters(AuthorizationRequest.fromHttpQueryString(Url(request).encodedQuery))
     }
+
+    fun initializeAuthorization(
+        authorizationRequest: AuthorizationRequest,
+        expiresIn: Duration,
+        selectedCredentials: Set<String>
+    ): VPresentationSession {
+        return super.initializeAuthorization(authorizationRequest, expiresIn).copy(selectedCredentialIds = selectedCredentials).also {
+            putSession(it.id, it)
+        }
+    }
 }