Skip to content

Commit

Permalink
Validate verifies the field JWTSupplier is not null before it is assi…
Browse files Browse the repository at this point in the history
…gned causing a `Parameter 'JWTSupplier JWT Token supplier cannot be null.'` exception. (#753)

Breaking IAMAWSProvider.GetAccessCredentials
See https://github.com/minio/minio-dotnet/blob/b332b78f47996ee40e2ed263b9374ad0e2663286/Minio/Credentials/IAMAWSProvider.cs#L93

This is a fix to set the field value before Validate().
  • Loading branch information
pjuarezd committed Mar 5, 2023
1 parent b332b78 commit bc8104f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Minio/Credentials/WebIdentityProvider.cs
Original file line number Diff line number Diff line change
Expand Up @@ -52,8 +52,8 @@ public override Task<AccessCredentials> GetCredentialsAsync()

internal WebIdentityProvider WithJWTSupplier(Func<JsonWebToken> f)
{
Validate();
JWTSupplier = (Func<JsonWebToken>)f.Clone();
Validate();
return this;
}

Expand Down

0 comments on commit bc8104f

Please sign in to comment.