Merge pull request #115 from dmcgowan/single-name-repo

Allow single name component repository names
This commit is contained in:
Stephen Day 2015-01-29 22:04:09 -08:00
commit 9e21347d75
3 changed files with 15 additions and 5 deletions

View file

@ -17,7 +17,7 @@ const (
// RepositoryNameMinComponents is the minimum number of slash-delimited
// components that a repository name must have
RepositoryNameMinComponents = 2
RepositoryNameMinComponents = 1
// RepositoryNameMaxComponents is the maximum number of slash-delimited
// components that a repository name must have
@ -41,9 +41,9 @@ var RepositoryNameComponentAnchoredRegexp = regexp.MustCompile(`^` + RepositoryN
// Looking path prefixes and s3 limitation of 1024, this should likely be
// around 512 bytes. 256 bytes might be more manageable.
// RepositoryNameRegexp builds on RepositoryNameComponentRegexp to allow 2 to
// RepositoryNameRegexp builds on RepositoryNameComponentRegexp to allow 1 to
// 5 path components, separated by a forward slash.
var RepositoryNameRegexp = regexp.MustCompile(`(?:` + RepositoryNameComponentRegexp.String() + `/){1,4}` + RepositoryNameComponentRegexp.String())
var RepositoryNameRegexp = regexp.MustCompile(`(?:` + RepositoryNameComponentRegexp.String() + `/){0,4}` + RepositoryNameComponentRegexp.String())
// TagNameRegexp matches valid tag names. From docker/docker:graph/tags.go.
var TagNameRegexp = regexp.MustCompile(`[\w][\w.-]{0,127}`)

View file

@ -9,6 +9,9 @@ func TestRepositoryNameRegexp(t *testing.T) {
input string
err error
}{
{
input: "short",
},
{
input: "simple/name",
},
@ -41,7 +44,6 @@ func TestRepositoryNameRegexp(t *testing.T) {
},
{
input: "asdf",
err: ErrRepositoryNameMissingComponents,
},
{
input: "asdf$$^/aa",
@ -58,7 +60,7 @@ func TestRepositoryNameRegexp(t *testing.T) {
},
{
input: "a",
err: ErrRepositoryNameMissingComponents,
err: ErrRepositoryNameComponentShort,
},
{
input: "a-/a/a/a",

View file

@ -51,6 +51,14 @@ func TestRouter(t *testing.T) {
RequestURI: "/v2/",
Vars: map[string]string{},
},
{
RouteName: RouteNameManifest,
RequestURI: "/v2/foo/manifests/bar",
Vars: map[string]string{
"name": "foo",
"tag": "bar",
},
},
{
RouteName: RouteNameManifest,
RequestURI: "/v2/foo/bar/manifests/tag",