Skip to content

Commit

Permalink
chore: remove commented code
Browse files Browse the repository at this point in the history
Signed-off-by: Shahram Kalantari <[email protected]>
  • Loading branch information
shahramk64 committed Oct 23, 2024
1 parent dfd5e9a commit a70ebef
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 2 deletions.
1 change: 0 additions & 1 deletion pkg/common/oras/authprovider/azure/azureidentity.go
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,6 @@ func (d *MIAuthProvider) Provide(ctx context.Context, artifact string) (provider
var options *azcontainerregistry.AuthenticationClientOptions
client, err := d.authClientFactory.CreateAuthClient(serverURL, options)
if err != nil {
// return provider.AuthConfig{}, re.ErrorCodeAuthDenied.NewError(re.AuthProvider, "", re.AzureWorkloadIdentityLink, err, "failed to create authentication client for container registry by azure managed identity token", re.HideStackTrace)
return provider.AuthConfig{}, re.ErrorCodeAuthDenied.WithError(err).WithDetail("failed to create authentication client for container registry by azure managed identity token")
}

Check warning on line 176 in pkg/common/oras/authprovider/azure/azureidentity.go

View check run for this annotation

Codecov / codecov/patch

pkg/common/oras/authprovider/azure/azureidentity.go#L175-L176

Added lines #L175 - L176 were not covered by tests

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,6 @@ func (d *WIAuthProvider) Provide(ctx context.Context, artifact string) (provider
var options *azcontainerregistry.AuthenticationClientOptions
client, err := d.authClientFactory.CreateAuthClient(serverURL, options)
if err != nil {
// return provider.AuthConfig{}, re.ErrorCodeAuthDenied.NewError(re.AuthProvider, "", re.AzureWorkloadIdentityLink, err, "failed to create authentication client for container registry", re.HideStackTrace)
return provider.AuthConfig{}, re.ErrorCodeAuthDenied.WithError(err).WithDetail("failed to create authentication client for container registry by azure managed identity token")
}

Check warning on line 178 in pkg/common/oras/authprovider/azure/azureworkloadidentity.go

View check run for this annotation

Codecov / codecov/patch

pkg/common/oras/authprovider/azure/azureworkloadidentity.go#L177-L178

Added lines #L177 - L178 were not covered by tests

Expand Down

0 comments on commit a70ebef

Please sign in to comment.