Skip to content

Commit 723de4a

Browse files
authored
fix: clean up warnings around default configuration for CrtHttpEngine (#438) (#440)
1 parent dc3cf24 commit 723de4a

File tree

1 file changed

+6
-2
lines changed
  • aws-runtime/http-client-engine-crt/common/src/aws/sdk/kotlin/runtime/http/engine/crt

1 file changed

+6
-2
lines changed

aws-runtime/http-client-engine-crt/common/src/aws/sdk/kotlin/runtime/http/engine/crt/CrtHttpEngine.kt

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -47,8 +47,12 @@ public class CrtHttpEngine(public val config: CrtHttpEngineConfig) : HttpClientE
4747
}
4848

4949
init {
50-
logger.warn { "CrtHttpEngine does not support HttpClientEngineConfig.socketReadTimeout(${config.socketReadTimeout}); ignoring" }
51-
logger.warn { "CrtHttpEngine does not support HttpClientEngineConfig.socketWriteTimeout(${config.socketWriteTimeout}); ignoring" }
50+
if (config.socketReadTimeout != CrtHttpEngineConfig.Default.socketReadTimeout) {
51+
logger.warn { "CrtHttpEngine does not support socketReadTimeout(${config.socketReadTimeout}); ignoring" }
52+
}
53+
if (config.socketWriteTimeout != CrtHttpEngineConfig.Default.socketWriteTimeout) {
54+
logger.warn { "CrtHttpEngine does not support socketWriteTimeout(${config.socketWriteTimeout}); ignoring" }
55+
}
5256
}
5357

5458
@OptIn(ExperimentalTime::class)

0 commit comments

Comments
 (0)