Refactor all pre-compiled regexp to package level vars
Addresses #8057 Docker-DCO-1.1-Signed-off-by: Phil Estes <estesp@linux.vnet.ibm.com>
This commit is contained in:
parent
a81b3ec0d7
commit
b7da79fd14
1 changed files with 2 additions and 2 deletions
|
@ -24,6 +24,8 @@ var (
|
||||||
ErrInvalidRepositoryName = errors.New("Invalid repository name (ex: \"registry.domain.tld/myrepos\")")
|
ErrInvalidRepositoryName = errors.New("Invalid repository name (ex: \"registry.domain.tld/myrepos\")")
|
||||||
errLoginRequired = errors.New("Authentication is required.")
|
errLoginRequired = errors.New("Authentication is required.")
|
||||||
validHex = regexp.MustCompile(`^([a-f0-9]{64})$`)
|
validHex = regexp.MustCompile(`^([a-f0-9]{64})$`)
|
||||||
|
validNamespace = regexp.MustCompile(`^([a-z0-9_]{4,30})$`)
|
||||||
|
validRepo = regexp.MustCompile(`^([a-z0-9-_.]+)$`)
|
||||||
)
|
)
|
||||||
|
|
||||||
type TimeoutType uint32
|
type TimeoutType uint32
|
||||||
|
@ -216,11 +218,9 @@ func validateRepositoryName(repositoryName string) error {
|
||||||
namespace = nameParts[0]
|
namespace = nameParts[0]
|
||||||
name = nameParts[1]
|
name = nameParts[1]
|
||||||
}
|
}
|
||||||
validNamespace := regexp.MustCompile(`^([a-z0-9_]{4,30})$`)
|
|
||||||
if !validNamespace.MatchString(namespace) {
|
if !validNamespace.MatchString(namespace) {
|
||||||
return fmt.Errorf("Invalid namespace name (%s), only [a-z0-9_] are allowed, size between 4 and 30", namespace)
|
return fmt.Errorf("Invalid namespace name (%s), only [a-z0-9_] are allowed, size between 4 and 30", namespace)
|
||||||
}
|
}
|
||||||
validRepo := regexp.MustCompile(`^([a-z0-9-_.]+)$`)
|
|
||||||
if !validRepo.MatchString(name) {
|
if !validRepo.MatchString(name) {
|
||||||
return fmt.Errorf("Invalid repository name (%s), only [a-z0-9-_.] are allowed", name)
|
return fmt.Errorf("Invalid repository name (%s), only [a-z0-9-_.] are allowed", name)
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in a new issue