fix: cert pinning failing for GET:/api-version (#2301) (#2302)

Co-authored-by: Mohamad Jaara <mohamad.jaara@wire.com>
wireapp · Dec 14, 2023 · 99c66b3 · 99c66b3
1 parent 568fde8
commit 99c66b3
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/logic/src/commonMain/kotlin/com/wire/kalium/logic/featureFlags/KaliumConfigs.kt b/logic/src/commonMain/kotlin/com/wire/kalium/logic/featureFlags/KaliumConfigs.kt
@@ -33,7 +33,7 @@ data class KaliumConfigs(
     val lowerKeyPackageLimits: Boolean = false,
     val lowerKeyingMaterialsUpdateThreshold: Boolean = false,
     val developmentApiEnabled: Boolean = false,
-    val ignoreSSLCertificatesForUnboundCalls: Boolean = true,
+    val ignoreSSLCertificatesForUnboundCalls: Boolean = false,
     val guestRoomLink: Boolean = true,
     val selfDeletingMessages: Boolean = true,
     val wipeOnCookieInvalid: Boolean = false,

diff --git a/network/src/commonJvmAndroid/kotlin/com/wire/kalium/network/HttpEngine.kt b/network/src/commonJvmAndroid/kotlin/com/wire/kalium/network/HttpEngine.kt
@@ -46,7 +46,7 @@ actual fun defaultHttpEngine(
     certificatePinning: CertificatePinning
 ): HttpClientEngine = OkHttp.create {
     OkhttpClientFactory {
-        if (certificatePinning.isNotEmpty()) {
+        if (certificatePinning.isNotEmpty() && !ignoreSSLCertificates) {
             val certPinner = CertificatePinner.Builder().apply {
                 certificatePinning.forEach { (cert, hosts) ->
                     hosts.forEach { host ->