Skip to content

Commit

Permalink
chore: address comments
Browse files Browse the repository at this point in the history
Signed-off-by: Shahram Kalantari <[email protected]>
  • Loading branch information
shahramk64 committed Sep 29, 2024
1 parent 5d4396c commit 4df7003
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion pkg/common/oras/authprovider/azure/azureworkloadidentity.go
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,8 @@ func (d *WIAuthProvider) Provide(ctx context.Context, artifact string) (provider

serverURL := "https://" + artifactHostName
// TODO: Consider adding authentication client options for multicloud scenarios
client, err := d.authClientFactory(serverURL, nil)
var options *azcontainerregistry.AuthenticationClientOptions
client, err := d.authClientFactory(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)

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

View check run for this annotation

Codecov / codecov/patch

pkg/common/oras/authprovider/azure/azureworkloadidentity.go#L161

Added line #L161 was not covered by tests
}
Expand Down

0 comments on commit 4df7003

Please sign in to comment.