Skip to content

Commit 609004d

Browse files
committed
address pr reviews
1 parent 5520c14 commit 609004d

File tree

2 files changed

+9
-11
lines changed

2 files changed

+9
-11
lines changed
Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,12 @@
11
public final class aws/smithy/kotlin/runtime/auth/awssigning/DefaultAwsSignerBuilder {
22
public fun <init> ()V
33
public final fun build ()Laws/smithy/kotlin/runtime/auth/awssigning/AwsSigner;
4-
public final fun withTelemetryProvider (Laws/smithy/kotlin/runtime/telemetry/TelemetryProvider;)Laws/smithy/kotlin/runtime/auth/awssigning/DefaultAwsSignerBuilder;
4+
public final fun getTelemetryProvider ()Laws/smithy/kotlin/runtime/telemetry/TelemetryProvider;
5+
public final fun setTelemetryProvider (Laws/smithy/kotlin/runtime/telemetry/TelemetryProvider;)V
56
}
67

78
public final class aws/smithy/kotlin/runtime/auth/awssigning/DefaultAwsSignerKt {
8-
public static final fun defaultAwsSigner (Lkotlin/jvm/functions/Function1;)Laws/smithy/kotlin/runtime/auth/awssigning/AwsSigner;
9+
public static final fun DefaultAwsSigner (Lkotlin/jvm/functions/Function1;)Laws/smithy/kotlin/runtime/auth/awssigning/AwsSigner;
910
public static final fun getDefaultAwsSigner ()Laws/smithy/kotlin/runtime/auth/awssigning/AwsSigner;
1011
}
1112

runtime/auth/aws-signing-default/common/src/aws/smithy/kotlin/runtime/auth/awssigning/DefaultAwsSigner.kt

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,14 @@ import kotlin.coroutines.coroutineContext
1515
/** The default implementation of [AwsSigner] */
1616
public val DefaultAwsSigner: AwsSigner = DefaultAwsSignerImpl()
1717

18-
public fun defaultAwsSigner(block: DefaultAwsSignerBuilder.() -> Unit): AwsSigner = DefaultAwsSignerBuilder().apply(block).build()
18+
/** Creates a customized instance of [AwsSigner] */
19+
@Suppress("ktlint:standard:function-naming")
20+
public fun DefaultAwsSigner(block: DefaultAwsSignerBuilder.() -> Unit): AwsSigner =
21+
DefaultAwsSignerBuilder().apply(block).build()
1922

20-
/**
21-
* Builder class allows customers to configure telemetry provider for observability when using the signer as a standalone component.
22-
**/
23+
/** A builder class for creating instances of [AwsSigner] using the default implementation */
2324
public class DefaultAwsSignerBuilder {
24-
private var telemetryProvider: TelemetryProvider? = null
25-
26-
public fun withTelemetryProvider(provider: TelemetryProvider): DefaultAwsSignerBuilder = apply {
27-
this.telemetryProvider = provider
28-
}
25+
public var telemetryProvider: TelemetryProvider? = null
2926

3027
public fun build(): AwsSigner = DefaultAwsSignerImpl(
3128
telemetryProvider = telemetryProvider,

0 commit comments

Comments
 (0)