Allow more product endpoint names to be valid

Product endpoint names can be started with digits or underscores.
Ericsson · Apr 19, 2018 · fc4d6b8 · fc4d6b8
1 parent 3826b81
commit fc4d6b8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 11 deletions.
diff --git a/libcodechecker/server/routing.py b/libcodechecker/server/routing.py
@@ -47,9 +47,7 @@ def is_valid_product_endpoint(uripart):
     if uripart in NON_PRODUCT_ENDPOINTS:
         return False
 
-    # Like programming variables: begin with letter and then letters, numbers,
-    # underscores.
-    pattern = r'^[A-Za-z][A-Za-z0-9_]*$'
+    pattern = r'^[A-Za-z0-9_]+$'
     if not re.match(pattern, uripart):
         return False
 

diff --git a/tests/functional/products/test_products.py b/tests/functional/products/test_products.py
@@ -98,14 +98,6 @@ def test_add_invalid_product(self):
         )
         product_cfg.connection = dbc
 
-        with self.assertRaises(RequestFailed):
-            product_cfg.endpoint = "_INVALID"
-            self._root_client.addProduct(product_cfg)
-
-        with self.assertRaises(RequestFailed):
-            product_cfg.endpoint = "0foobar"
-            self._root_client.addProduct(product_cfg)
-
         with self.assertRaises(RequestFailed):
             product_cfg.endpoint = "$$$$$$$"
             self._root_client.addProduct(product_cfg)