diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2019-11-08 22:06:34 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-11-08 22:06:34 +0100 |
commit | f456ce90f966f2eefdfc27ca83541ceacc4a298d (patch) | |
tree | 24f39eb6404de474cb2e2d41acc2a3ace8410ce5 /cmd/podman/shared/funcs.go | |
parent | 651d6ebe5274a892ce64819bb007080895f98321 (diff) | |
parent | d7c0f968ca60994306c8c76cd8e4e0a677fe9ada (diff) | |
download | podman-f456ce90f966f2eefdfc27ca83541ceacc4a298d.tar.gz podman-f456ce90f966f2eefdfc27ca83541ceacc4a298d.tar.bz2 podman-f456ce90f966f2eefdfc27ca83541ceacc4a298d.zip |
Merge pull request #4337 from QiWang19/check_auth_path
fix bug check nonexist authfile
Diffstat (limited to 'cmd/podman/shared/funcs.go')
-rw-r--r-- | cmd/podman/shared/funcs.go | 21 |
1 files changed, 8 insertions, 13 deletions
diff --git a/cmd/podman/shared/funcs.go b/cmd/podman/shared/funcs.go index 9362e8e9b..404d0f288 100644 --- a/cmd/podman/shared/funcs.go +++ b/cmd/podman/shared/funcs.go @@ -6,24 +6,19 @@ import ( "path/filepath" "strings" - "github.com/containers/libpod/pkg/util" + "github.com/containers/image/v5/types" + "github.com/containers/libpod/libpod/image" "github.com/google/shlex" + "github.com/pkg/errors" ) -func GetAuthFile(authfile string) string { +func GetSystemContext(authfile string) (*types.SystemContext, error) { if authfile != "" { - return authfile - } - - authfile = os.Getenv("REGISTRY_AUTH_FILE") - if authfile != "" { - return authfile - } - - if runtimeDir, err := util.GetRuntimeDir(); err == nil { - return filepath.Join(runtimeDir, "containers/auth.json") + if _, err := os.Stat(authfile); err != nil { + return nil, errors.Wrapf(err, "error checking authfile path %s", authfile) + } } - return "" + return image.GetSystemContext("", authfile, false), nil } func substituteCommand(cmd string) (string, error) { |