diff options
author | OpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com> | 2020-05-21 19:55:58 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-21 19:55:58 +0200 |
commit | 9d3b46624b4eb3b6eeea38b5b1057914a9ff2baf (patch) | |
tree | 54fbc064fe4f7723407ed8d217148c0b1be4cb7a /libpod | |
parent | c3b6e623bba748fcfdae4f755e04c7185a85e79c (diff) | |
parent | 0d0565f55ee3399f5c2c4e4b73acb732b424f3db (diff) | |
download | podman-9d3b46624b4eb3b6eeea38b5b1057914a9ff2baf.tar.gz podman-9d3b46624b4eb3b6eeea38b5b1057914a9ff2baf.tar.bz2 podman-9d3b46624b4eb3b6eeea38b5b1057914a9ff2baf.zip |
Merge pull request #6323 from rhatdan/shrink
Remove github.com/libpod/libpod from cmd/pkg/podman
Diffstat (limited to 'libpod')
-rw-r--r-- | libpod/define/config.go | 10 | ||||
-rw-r--r-- | libpod/options.go | 26 |
2 files changed, 18 insertions, 18 deletions
diff --git a/libpod/define/config.go b/libpod/define/config.go index 692eafb04..5ca4da4af 100644 --- a/libpod/define/config.go +++ b/libpod/define/config.go @@ -3,6 +3,9 @@ package define import ( "bufio" "io" + "regexp" + + "github.com/pkg/errors" ) var ( @@ -10,6 +13,13 @@ var ( DefaultSHMLockPath = "/libpod_lock" // DefaultRootlessSHMLockPath is the default path for rootless SHM locks DefaultRootlessSHMLockPath = "/libpod_rootless_lock" + + // NameRegex is a regular expression to validate container/pod names. + // This must NOT be changed from outside of Libpod. It should be a + // constant, but Go won't let us do that. + NameRegex = regexp.MustCompile("^[a-zA-Z0-9][a-zA-Z0-9_.-]*$") + // RegexError is thrown in presence of an invalid container/pod name. + RegexError = errors.Wrapf(ErrInvalidArg, "names must match [a-zA-Z0-9][a-zA-Z0-9_.-]*") ) const ( diff --git a/libpod/options.go b/libpod/options.go index 05241baf3..ff5e29335 100644 --- a/libpod/options.go +++ b/libpod/options.go @@ -4,7 +4,6 @@ import ( "net" "os" "path/filepath" - "regexp" "syscall" "github.com/containers/common/pkg/config" @@ -20,15 +19,6 @@ import ( "github.com/pkg/errors" ) -var ( - // NameRegex is a regular expression to validate container/pod names. - // This must NOT be changed from outside of Libpod. It should be a - // constant, but Go won't let us do that. - NameRegex = regexp.MustCompile("^[a-zA-Z0-9][a-zA-Z0-9_.-]*$") - // RegexError is thrown in presence of an invalid container/pod name. - RegexError = errors.Wrapf(define.ErrInvalidArg, "names must match [a-zA-Z0-9][a-zA-Z0-9_.-]*") -) - // Runtime Creation Options // WithStorageConfig uses the given configuration to set up container storage. @@ -665,8 +655,8 @@ func WithName(name string) CtrCreateOption { } // Check the name against a regex - if !NameRegex.MatchString(name) { - return RegexError + if !define.NameRegex.MatchString(name) { + return define.RegexError } ctr.config.Name = name @@ -1383,8 +1373,8 @@ func WithVolumeName(name string) VolumeCreateOption { } // Check the name against a regex - if !NameRegex.MatchString(name) { - return RegexError + if !define.NameRegex.MatchString(name) { + return define.RegexError } volume.config.Name = name @@ -1502,8 +1492,8 @@ func WithPodName(name string) PodCreateOption { } // Check the name against a regex - if !NameRegex.MatchString(name) { - return RegexError + if !define.NameRegex.MatchString(name) { + return define.RegexError } pod.config.Name = name @@ -1520,8 +1510,8 @@ func WithPodHostname(hostname string) PodCreateOption { } // Check the hostname against a regex - if !NameRegex.MatchString(hostname) { - return RegexError + if !define.NameRegex.MatchString(hostname) { + return define.RegexError } pod.config.Hostname = hostname |