Ensure that we limit the length of repository names

Until now, they'd simply be truncated by the database. Now, we properly check their lengths.

Fixes https://jira.coreos.com/browse/QUAY-963

test/registry/protocol_v1.py

@@ -21,7 +21,8 @@ class V1Protocol(RegistryProtocol):
       Failures.UNAUTHENTICATED: 401,
       Failures.UNAUTHORIZED: 403,
       Failures.APP_REPOSITORY: 405,
-      Failures.INVALID_REPOSITORY: 404,
+      Failures.SLASH_REPOSITORY: 404,
+      Failures.INVALID_REPOSITORY: 400,
       Failures.DISALLOWED_LIBRARY_NAMESPACE: 400,
       Failures.NAMESPACE_DISABLED: 400,
     },

test/registry/protocol_v2.py

@@ -28,6 +28,7 @@ class V2Protocol(RegistryProtocol):
       Failures.APP_REPOSITORY: 405,
       Failures.ANONYMOUS_NOT_ALLOWED: 401,
       Failures.INVALID_REPOSITORY: 400,
+      Failures.SLASH_REPOSITORY: 400,
       Failures.NAMESPACE_DISABLED: 400,
     },
     V2ProtocolSteps.MOUNT_BLOB: {

test/registry/protocols.py

@@ -41,6 +41,7 @@ class Failures(Enum):
   UNAUTHORIZED = 'unauthorized'
   INVALID_REGISTRY = 'invalid-registry'
   INVALID_REPOSITORY = 'invalid-repository'
+  SLASH_REPOSITORY = 'slash-repository'
   APP_REPOSITORY = 'app-repository'
   UNKNOWN_TAG = 'unknown-tag'
   ANONYMOUS_NOT_ALLOWED = 'anonymous-not-allowed'

test/registry/registry_tests.py

@@ -362,15 +362,25 @@ def test_image_replication(pusher, basic_images, liveserver_session, app_reloade
       assert r.text == 'OK'
 
 
-def test_push_reponame_with_slashes(pusher, basic_images, liveserver_session, app_reloader):
-  """ Test: Attempt to add a repository name with slashes. This should fail as we do not
-      support it.
+@pytest.mark.parametrize('repo_name, expected_failure', [
+  ('something', None),
+  ('some/slash', Failures.SLASH_REPOSITORY),
+  pytest.param('x' * 255, None, id='Valid long name'),
+  pytest.param('x' * 256, Failures.INVALID_REPOSITORY, id='Name too long'),
+])
+def test_push_reponame(repo_name, expected_failure, pusher, puller, basic_images,
+                       liveserver_session, app_reloader):
+  """ Test: Attempt to add a repository with various names.
   """
   credentials = ('devtable', 'password')
 
-  pusher.push(liveserver_session, 'devtable', 'some/slash', 'latest', basic_images,
+  pusher.push(liveserver_session, 'devtable', repo_name, 'latest', basic_images,
               credentials=credentials,
-              expected_failure=Failures.INVALID_REPOSITORY)
+              expected_failure=expected_failure)
+
+  if expected_failure is None:
+    puller.pull(liveserver_session, 'devtable', repo_name, 'latest', basic_images,
+                credentials=credentials)
 
 
 @pytest.mark.parametrize('tag_name, expected_failure', [