diff options
author | Miloslav Trmač <mitr@redhat.com> | 2021-10-21 21:31:22 +0200 |
---|---|---|
committer | Miloslav Trmač <mitr@redhat.com> | 2021-12-10 18:16:27 +0100 |
commit | 3cfefa1248feb9de8041b9fc67987b508d4c3fcb (patch) | |
tree | 3d426ef42c91b6d32c5f2607a1dadf9929ec3c5d /pkg/bindings/images/build.go | |
parent | d79414c54ff89c3deea84e2ac600525744fc75c5 (diff) | |
download | podman-3cfefa1248feb9de8041b9fc67987b508d4c3fcb.tar.gz podman-3cfefa1248feb9de8041b9fc67987b508d4c3fcb.tar.bz2 podman-3cfefa1248feb9de8041b9fc67987b508d4c3fcb.zip |
Remove the authfile parameter of MakeXRegistryAuthHeader
Having a parameter that modifies the provides types.SystemContext
seems rather unexpected and risky to have around - and the only
user of that is actually a no-op, others only provide a nil
SystemContext; so, remove that option and simplify (well, somewhat;
many callers now have extra &types.SystemContext{AuthFilePath}
boilerplate; at least that's consistent with that code carrying
a TODO to create a larger-scope SystemContext).
Should not change behavior.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Diffstat (limited to 'pkg/bindings/images/build.go')
-rw-r--r-- | pkg/bindings/images/build.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/pkg/bindings/images/build.go b/pkg/bindings/images/build.go index fece5e9d0..7bca43132 100644 --- a/pkg/bindings/images/build.go +++ b/pkg/bindings/images/build.go @@ -294,7 +294,7 @@ func Build(ctx context.Context, containerFiles []string, options entities.BuildO err error ) if options.SystemContext != nil && options.SystemContext.DockerAuthConfig != nil { - headers, err = auth.MakeXRegistryAuthHeader(options.SystemContext, options.SystemContext.AuthFilePath, options.SystemContext.DockerAuthConfig.Username, options.SystemContext.DockerAuthConfig.Password) + headers, err = auth.MakeXRegistryAuthHeader(options.SystemContext, options.SystemContext.DockerAuthConfig.Username, options.SystemContext.DockerAuthConfig.Password) } else { headers, err = auth.MakeXRegistryConfigHeader(options.SystemContext, "", "") } |