Relax route name restriction: add dashes

Author: asvetlov

CHANGES.txt

@@ -1,6 +1,13 @@
 CHANGES
 =======
 
+0.18.4 (13-11-2015)
+-------------------
+
+- Relax rule for router names again by adding dash to allowed
+  characters: they may contain identifiers, dashes, dots and columns
+
+
 0.18.3 (25-10-2015)
 -------------------
 

aiohttp/web_urldispatcher.py

@@ -389,7 +389,7 @@ class UrlDispatcher(AbstractRouter, collections.abc.Mapping):
         r'^\{(?P<var>[a-zA-Z][_a-zA-Z0-9]*):(?P<re>.+)\}$')
     GOOD = r'[^{}/]+'
     ROUTE_RE = re.compile(r'(\{[_a-zA-Z][^{}]*(?:\{[^{}]*\}[^{}]*)*\})')
-    NAME_SPLIT_RE = re.compile('[.:]')
+    NAME_SPLIT_RE = re.compile('[.:-]')
 
     METHODS = {hdrs.METH_ANY, hdrs.METH_POST,
                hdrs.METH_GET, hdrs.METH_PUT, hdrs.METH_DELETE,
@@ -449,10 +449,10 @@ def register_route(self, route):
             parts = self.NAME_SPLIT_RE.split(name)
             for part in parts:
                 if not part.isidentifier() or keyword.iskeyword(part):
-                    raise ValueError('Incorrect route name value, '
-                                     'Route name should be a sequence of '
+                    raise ValueError('Incorrect route name {!r}, '
+                                     'the name should be a sequence of '
                                      'python identifiers separated '
-                                     'by dot or column')
+                                     'by dash, dot or column'.format(name))
             if name in self._routes:
                 raise ValueError('Duplicate {!r}, '
                                  'already handled by {!r}'

tests/test_urldispatch.py

@@ -83,7 +83,7 @@ def test_register_route_checks(self):
         route = PlainRoute('GET', handler, 'return', '/handler/to/path')
         self.assertRaises(ValueError, self.router.register_route, route)
 
-        route = PlainRoute('GET', handler, 'test.test:test',
+        route = PlainRoute('GET', handler, 'test.test:test-test',
                            '/handler/to/path')
         self.router.register_route(route)