Skip to content

Commit

Permalink
fix-kms-connections-config: Set kms max acquisition timeout configura…
Browse files Browse the repository at this point in the history
…tion (#284)
  • Loading branch information
galales authored Oct 6, 2023
1 parent 5c373c0 commit b211c7f
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 2 deletions.
1 change: 1 addition & 0 deletions signer/src/main/resources/reference.conf
Original file line number Diff line number Diff line change
Expand Up @@ -2,4 +2,5 @@

interop-commons.kms {
max-concurrency = 50
max-acquisition-timeout = "10 seconds"
}
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ package it.pagopa.interop.commons.signer
import com.typesafe.config.{Config, ConfigFactory}

import java.net.URI
import java.time.Duration

final case class VaultConfig(address: String, token: String, sslEnabled: Boolean, signatureRoute: String) {
def encryptionEndpoint(keyId: String) = new URI(s"$address/$signatureRoute/$keyId").normalize().toString
Expand All @@ -14,6 +15,7 @@ object SignerConfiguration {

private val config: Config = ConfigFactory.load()

val maxConcurrency: Int = config.getInt("interop-commons.kms.max-concurrency")
val maxConcurrency: Int = config.getInt("interop-commons.kms.max-concurrency")
val maxAcquisitionTimeout: Duration = config.getDuration("interop-commons.kms.max-acquisition-timeout")

}
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,11 @@ final class KMSSignerService(blockingEc: ExecutionContextExecutor) extends Signe
implicit val ec: ExecutionContext = blockingEc

private val asyncHttpClient: SdkAsyncHttpClient =
NettyNioAsyncHttpClient.builder().maxConcurrency(SignerConfiguration.maxConcurrency).build()
NettyNioAsyncHttpClient
.builder()
.connectionAcquisitionTimeout(SignerConfiguration.maxAcquisitionTimeout)
.maxConcurrency(SignerConfiguration.maxConcurrency)
.build()
private val asyncConfiguration: ClientAsyncConfiguration =
ClientAsyncConfiguration
.builder()
Expand Down

0 comments on commit b211c7f

Please sign in to comment.