Merge pull request #57 from piercetrey-figure/piercetrey-figure/sc-26…

…8894/signer-flexibility-improvements flexibility improvements for signing transactions
provenance-io · Nov 17, 2023 · 9cdbb35 · 9cdbb35
2 parents 7c600a7 + 5b9309e
commit 9cdbb35
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 9 deletions.
diff --git a/build.gradle.kts b/build.gradle.kts
@@ -67,6 +67,10 @@ subprojects {
     configurations.forEach { it.exclude("org.slf4j", "slf4j-api") }
     val artifactName = "pb-grpc-$name-kotlin"
 
+    tasks.withType<PublishToMavenLocal> {
+        signing.isRequired = false
+    }
+
     configure<PublishingExtension> {
         publications {
             create<MavenPublication>("maven") {

diff --git a/client-common/src/main/kotlin/io/provenance/client/grpc/BaseReq.kt b/client-common/src/main/kotlin/io/provenance/client/grpc/BaseReq.kt
@@ -21,11 +21,16 @@ interface Signer {
     fun sign(data: ByteArray): ByteArray
 }
 
+typealias CustomUnpackAccount = (Any.() -> Auth.BaseAccount)
+typealias CustomizeSignerInfo = (SignerInfo.() -> SignerInfo)
 data class BaseReqSigner(
     val signer: Signer,
     val sequenceOffset: Int = 0,
-    val account: Auth.BaseAccount? = null
+    val account: Auth.BaseAccount? = null,
+    private val customizeSignerInfo: CustomizeSignerInfo? = null,
+    val unpackAccount: CustomUnpackAccount? = null,
 ) {
+
     fun buildSignerInfo(): SignerInfo =
         SignerInfo.newBuilder()
             .setPublicKey(Any.pack(this.signer.pubKey(), ""))
@@ -34,7 +39,12 @@ data class BaseReqSigner(
                     .setSingle(Single.newBuilder().setModeValue(SignMode.SIGN_MODE_DIRECT_VALUE))
             )
             .setSequence(this.account!!.sequence + this.sequenceOffset)
-            .build()
+            .build().let {
+                customizeSignerInfo?.invoke(it) ?: it
+            }
+
+    fun withCustomizeSignerInfo(customizeSignerInfo: CustomizeSignerInfo) = copy(customizeSignerInfo = customizeSignerInfo)
+    fun withUnpackAccount(unpackAccount: CustomUnpackAccount) = copy(unpackAccount = unpackAccount)
 }
 
 data class BaseReq(

diff --git a/client-coroutines/src/main/kotlin/io/provenance/client/coroutines/PbCoroutinesClient.kt b/client-coroutines/src/main/kotlin/io/provenance/client/coroutines/PbCoroutinesClient.kt
@@ -1,6 +1,8 @@
 package io.provenance.client.coroutines
 
+import com.google.protobuf.Any
 import com.google.protobuf.ByteString
+import cosmos.auth.v1beta1.Auth.BaseAccount
 import cosmos.auth.v1beta1.QueryGrpcKt
 import cosmos.auth.v1beta1.QueryOuterClass
 import cosmos.base.tendermint.v1beta1.getLatestBlockRequest
@@ -81,10 +83,8 @@ open class PbCoroutinesClient(
         feePayer: String? = null,
     ): BaseReq =
         signers.map {
-            BaseReqSigner(
-                signer = it.signer,
-                sequenceOffset = it.sequenceOffset,
-                account = it.account ?: this.authClient.getBaseAccount(it.signer.address())
+            it.copy(
+                account = it.account ?: this.authClient.getBaseAccount(it.signer.address(), unpackAccount = it.unpackAccount)
             )
         }.let {
             BaseReq(
@@ -203,12 +203,13 @@ open class PbCoroutinesClient(
  * See [Accounts](https://github.com/FigureTechnologies/service-wallet/blob/v45/pb-client/src/main/kotlin/com/figure/wallet/pbclient/client/grpc/Accounts.kt#L18).
  *
  * @param bech32Address The bech32 address to fetch.
- * @return [cosmos.auth.v1beta1.Auth.BaseAccount] or throw [IllegalArgumentException] if the account type is not supported.
+ * @return [BaseAccount] or throw [IllegalArgumentException] if the account type is not supported.
  */
-suspend fun QueryGrpcKt.QueryCoroutineStub.getBaseAccount(bech32Address: String): cosmos.auth.v1beta1.Auth.BaseAccount =
+suspend fun QueryGrpcKt.QueryCoroutineStub.getBaseAccount(bech32Address: String, unpackAccount: (Any.() -> BaseAccount)? = null): BaseAccount =
     account(QueryOuterClass.QueryAccountRequest.newBuilder().setAddress(bech32Address).build()).account.run {
         when {
-            this.`is`(cosmos.auth.v1beta1.Auth.BaseAccount::class.java) -> unpack(cosmos.auth.v1beta1.Auth.BaseAccount::class.java)
+            unpackAccount != null -> unpackAccount()
+            this.`is`(BaseAccount::class.java) -> unpack(BaseAccount::class.java)
             else -> throw IllegalArgumentException("Account type not handled:$typeUrl")
         }
     }