all platform signing provider signers should be attestable

a-sit-plus · Oct 29, 2024 · 033a466 · 033a466
1 parent 78a3742
commit 033a466
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/supreme/src/androidMain/kotlin/at/asitplus/signum/supreme/os/AndroidKeyStoreProvider.kt b/supreme/src/androidMain/kotlin/at/asitplus/signum/supreme/os/AndroidKeyStoreProvider.kt
@@ -291,7 +291,7 @@ sealed class AndroidKeystoreSigner private constructor(
     val keyInfo: KeyInfo,
     private val config: AndroidSignerConfiguration,
     final override val attestation: AndroidKeystoreAttestation?
-) : PlatformSigningProviderSigner<AndroidSignerSigningConfiguration>, SignerI.Attestable<AndroidKeystoreAttestation> {
+) : PlatformSigningProviderSigner<AndroidSignerSigningConfiguration, AndroidKeystoreAttestation> {
 
     final override val mayRequireUserUnlock: Boolean get() = this.needsAuthentication
 

diff --git a/supreme/src/commonMain/kotlin/at/asitplus/signum/supreme/os/SigningProvider.kt b/supreme/src/commonMain/kotlin/at/asitplus/signum/supreme/os/SigningProvider.kt
@@ -2,6 +2,7 @@ package at.asitplus.signum.supreme.os
 
 import at.asitplus.KmmResult
 import at.asitplus.catching
+import at.asitplus.signum.indispensable.Attestation
 import at.asitplus.signum.indispensable.Digest
 import at.asitplus.signum.indispensable.RSAPadding
 import at.asitplus.signum.supreme.SignatureResult
@@ -139,8 +140,9 @@ open class PlatformSigningProviderSignerSigningConfigurationBase internal constr
     open val unlockPrompt = childOrDefault(::UnlockPromptConfiguration)
 }
 
-interface PlatformSigningProviderSigner<SigningConfiguration: PlatformSigningProviderSignerSigningConfigurationBase>
-    : Signer.WithAlias {
+interface PlatformSigningProviderSigner
+    <SigningConfiguration: PlatformSigningProviderSignerSigningConfigurationBase, AttestationT: Attestation>
+    : Signer.WithAlias, Signer.Attestable<AttestationT> {
 
     suspend fun trySetupUninterruptedSigning(configure: DSLConfigureFn<SigningConfiguration> = null) : KmmResult<Unit> = KmmResult.success(Unit)
     override suspend fun trySetupUninterruptedSigning() = trySetupUninterruptedSigning(null)
@@ -160,7 +162,7 @@ internal expect fun getPlatformSigningProvider(configure: DSLConfigureFn<Platfor
 
 /** KT-71089 workaround
  * @see PlatformSigningProvider */
-interface PlatformSigningProviderI<out SignerT: PlatformSigningProviderSigner<*>,
+interface PlatformSigningProviderI<out SignerT: PlatformSigningProviderSigner<*,*>,
         out SignerConfigT: PlatformSignerConfigurationBase,
         out KeyConfigT: PlatformSigningKeyConfigurationBase<*>>
     : SigningProviderI<SignerT, SignerConfigT, KeyConfigT> {

diff --git a/supreme/src/iosMain/kotlin/at/asitplus/signum/supreme/os/IosKeychainProvider.kt b/supreme/src/iosMain/kotlin/at/asitplus/signum/supreme/os/IosKeychainProvider.kt
@@ -179,7 +179,7 @@ typealias IosSignerSigningConfiguration = PlatformSigningProviderSignerSigningCo
 sealed class IosSigner(final override val alias: String,
                        private val metadata: IosKeyMetadata,
                        private val signerConfig: IosSignerConfiguration)
-    : PlatformSigningProviderSigner<IosSignerSigningConfiguration>, Signer.Attestable<IosHomebrewAttestation> {
+    : PlatformSigningProviderSigner<IosSignerSigningConfiguration, IosHomebrewAttestation> {
 
     override val mayRequireUserUnlock get() = needsAuthentication
     val needsAuthentication get() = metadata.needsUnlock