Remove changes to IdentityProvider & IdentityProviderChain. Override CredentialsProviderChain toString

Author: 0marperez

runtime/auth/aws-credentials/api/aws-credentials.api

@@ -2,7 +2,6 @@ public final class aws/smithy/kotlin/runtime/auth/awscredentials/CachedCredentia
 	public synthetic fun <init> (Laws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider;JJLaws/smithy/kotlin/runtime/time/Clock;ILkotlin/jvm/internal/DefaultConstructorMarker;)V
 	public synthetic fun <init> (Laws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider;JJLaws/smithy/kotlin/runtime/time/Clock;Lkotlin/jvm/internal/DefaultConstructorMarker;)V
 	public fun close ()V
-	public fun getProviderName ()Ljava/lang/String;
 	public fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }
 
@@ -15,10 +14,6 @@ public final class aws/smithy/kotlin/runtime/auth/awscredentials/CachedCredentia
 public abstract interface class aws/smithy/kotlin/runtime/auth/awscredentials/CloseableCredentialsProvider : aws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider, java/io/Closeable {
 }
 
-public final class aws/smithy/kotlin/runtime/auth/awscredentials/CloseableCredentialsProvider$DefaultImpls {
-	public static fun getProviderName (Laws/smithy/kotlin/runtime/auth/awscredentials/CloseableCredentialsProvider;)Ljava/lang/String;
-}
-
 public abstract interface class aws/smithy/kotlin/runtime/auth/awscredentials/Credentials : aws/smithy/kotlin/runtime/identity/Identity {
 	public static final field Companion Laws/smithy/kotlin/runtime/auth/awscredentials/Credentials$Companion;
 	public abstract fun getAccessKeyId ()Ljava/lang/String;
@@ -46,14 +41,11 @@ public abstract interface class aws/smithy/kotlin/runtime/auth/awscredentials/Cr
 	public abstract fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }
 
-public final class aws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider$DefaultImpls {
-	public static fun getProviderName (Laws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider;)Ljava/lang/String;
-}
-
 public final class aws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProviderChain : aws/smithy/kotlin/runtime/identity/IdentityProviderChain, aws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider {
 	public fun <init> (Ljava/util/List;)V
 	public fun <init> ([Laws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProvider;)V
 	public fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
+	public fun toString ()Ljava/lang/String;
 }
 
 public abstract interface class aws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProviderConfig {

runtime/auth/aws-credentials/common/src/aws/smithy/kotlin/runtime/auth/awscredentials/CredentialsProviderChain.kt

@@ -21,4 +21,6 @@ public class CredentialsProviderChain(vararg providers: CredentialsProvider) :
     public constructor(providers: List<CredentialsProvider>) : this(*providers.toTypedArray())
 
     override suspend fun resolve(attributes: Attributes): Credentials = super.resolve(attributes)
+
+    override fun toString(): String = providers.joinToString(" -> ")
 }

runtime/auth/http-auth/api/http-auth.api

@@ -18,7 +18,6 @@ public final class aws/smithy/kotlin/runtime/http/auth/AnonymousIdentity : aws/s
 
 public final class aws/smithy/kotlin/runtime/http/auth/AnonymousIdentityProvider : aws/smithy/kotlin/runtime/identity/IdentityProvider {
 	public static final field INSTANCE Laws/smithy/kotlin/runtime/http/auth/AnonymousIdentityProvider;
-	public fun getProviderName ()Ljava/lang/String;
 	public fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }
 
@@ -37,10 +36,6 @@ public abstract interface class aws/smithy/kotlin/runtime/http/auth/BearerTokenP
 	public abstract fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }
 
-public final class aws/smithy/kotlin/runtime/http/auth/BearerTokenProvider$DefaultImpls {
-	public static fun getProviderName (Laws/smithy/kotlin/runtime/http/auth/BearerTokenProvider;)Ljava/lang/String;
-}
-
 public final class aws/smithy/kotlin/runtime/http/auth/BearerTokenProviderChain : aws/smithy/kotlin/runtime/identity/IdentityProviderChain, aws/smithy/kotlin/runtime/http/auth/BearerTokenProvider {
 	public fun <init> (Ljava/util/List;)V
 	public fun <init> ([Laws/smithy/kotlin/runtime/http/auth/BearerTokenProvider;)V
@@ -64,7 +59,3 @@ public final class aws/smithy/kotlin/runtime/http/auth/BearerTokenSigner : aws/s
 public abstract interface class aws/smithy/kotlin/runtime/http/auth/CloseableBearerTokenProvider : aws/smithy/kotlin/runtime/http/auth/BearerTokenProvider, java/io/Closeable {
 }
 
-public final class aws/smithy/kotlin/runtime/http/auth/CloseableBearerTokenProvider$DefaultImpls {
-	public static fun getProviderName (Laws/smithy/kotlin/runtime/http/auth/CloseableBearerTokenProvider;)Ljava/lang/String;
-}
-

runtime/auth/identity-api/api/identity-api.api

@@ -40,10 +40,6 @@ public abstract interface class aws/smithy/kotlin/runtime/auth/AuthSchemeProvide
 public abstract interface class aws/smithy/kotlin/runtime/identity/CloseableIdentityProvider : aws/smithy/kotlin/runtime/identity/IdentityProvider, java/io/Closeable {
 }
 
-public final class aws/smithy/kotlin/runtime/identity/CloseableIdentityProvider$DefaultImpls {
-	public static fun getProviderName (Laws/smithy/kotlin/runtime/identity/CloseableIdentityProvider;)Ljava/lang/String;
-}
-
 public abstract interface class aws/smithy/kotlin/runtime/identity/Identity {
 	public abstract fun getAttributes ()Laws/smithy/kotlin/runtime/collections/Attributes;
 	public abstract fun getExpiration ()Laws/smithy/kotlin/runtime/time/Instant;
@@ -59,19 +55,16 @@ public final class aws/smithy/kotlin/runtime/identity/IdentityAttributesKt {
 }
 
 public abstract interface class aws/smithy/kotlin/runtime/identity/IdentityProvider {
-	public abstract fun getProviderName ()Ljava/lang/String;
 	public abstract fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 }
 
 public final class aws/smithy/kotlin/runtime/identity/IdentityProvider$DefaultImpls {
-	public static fun getProviderName (Laws/smithy/kotlin/runtime/identity/IdentityProvider;)Ljava/lang/String;
 	public static synthetic fun resolve$default (Laws/smithy/kotlin/runtime/identity/IdentityProvider;Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;ILjava/lang/Object;)Ljava/lang/Object;
 }
 
 public abstract class aws/smithy/kotlin/runtime/identity/IdentityProviderChain : aws/smithy/kotlin/runtime/identity/CloseableIdentityProvider {
 	public fun <init> ([Laws/smithy/kotlin/runtime/identity/IdentityProvider;)V
 	public fun close ()V
-	public fun getProviderName ()Ljava/lang/String;
 	protected final fun getProviders ()[Laws/smithy/kotlin/runtime/identity/IdentityProvider;
 	public fun resolve (Laws/smithy/kotlin/runtime/collections/Attributes;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
 	public fun toString ()Ljava/lang/String;

runtime/auth/identity-api/common/src/aws/smithy/kotlin/runtime/identity/IdentityProvider.kt

@@ -20,15 +20,6 @@ public interface IdentityProvider {
      * @return An [Identity] that can be used to connect to the service
      */
     public suspend fun resolve(attributes: Attributes = emptyAttributes()): Identity
-
-    /**
-     * The name of the identity provider.
-     *
-     * It ensures that the name is still available even if the identity provider is wrapped or decorated.
-     * By default, it returns `null`, but implementations can override it to provide a meaningful name.
-     */
-    public val providerName: String?
-        get() = null
 }
 
 /**

runtime/auth/identity-api/common/src/aws/smithy/kotlin/runtime/identity/IdentityProviderChain.kt

@@ -29,7 +29,7 @@ public abstract class IdentityProviderChain<P : IdentityProvider, I : Identity>(
         require(providers.isNotEmpty()) { "at least one provider must be in the chain" }
     }
     override fun toString(): String =
-        (listOf(this) + providers).map { it.providerName ?: it::class.simpleName }.joinToString(" -> ")
+        (listOf(this) + providers).map { it::class.simpleName }.joinToString(" -> ")
 
     override suspend fun resolve(attributes: Attributes): I = withSpan<IdentityProviderChain<*, *>, I>("ResolveIdentityChain") {
         val logger = coroutineContext.logger<IdentityProviderChain<*, *>>()