Fix path_template variable parsing containing dots

see plotly#2437

dash/_pages.py

@@ -3,7 +3,6 @@
 from os.path import isfile, join
 import collections
 from urllib.parse import parse_qs
-from fnmatch import fnmatch
 import re
 import flask
 
@@ -101,23 +100,14 @@ def _parse_path_variables(pathname, path_template):
          returns **{"asset_id": "a100"}
     """
 
-    # parse variable definitions e.g. <var_name> from template
-    # and create pattern to match
-    wildcard_pattern = re.sub("<.*?>", "*", path_template)
-    var_pattern = re.sub("<.*?>", "(.*)", path_template)
-
-    # check that static sections of the pathname match the template
-    if not fnmatch(pathname, wildcard_pattern):
-        return None
-
-    # parse variable names e.g. var_name from template
-    var_names = re.findall("<(.*?)>", path_template)
-
-    # parse variables from path
-    variables = re.findall(var_pattern, pathname)
-    variables = variables[0] if isinstance(variables[0], tuple) else variables
-
-    return dict(zip(var_names, variables))
+    pattern = re.compile(
+        re.sub(
+            "<(?P<var>.*?)>", r"(?P<\g<var>>.*)",
+            re.escape(path_template)
+        )
+    )
+    match = pattern.match(pathname)
+    return match.groupdict() if match else None
 
 
 def _create_redirect_function(redirect_to):

tests/unit/test_path_template.py

@@ -0,0 +1,22 @@
+from dash._pages import _parse_path_variables
+
+
+def test_path_template_general():
+    path_template = "some/<var>/static"
+
+    assert _parse_path_variables("some/value/static", path_template) == {'var': 'value'}
+
+
+def test_path_template():
+    path_template = "<foo>/<bar>"
+    assert _parse_path_variables("one/two", path_template) == {'foo': 'one', 'bar': 'two'}
+
+
+def test_path_template_dots():
+    path_template = "<foo>.<bar>"
+    assert _parse_path_variables("one.two", path_template) == {'foo': 'one', 'bar': 'two'}
+
+
+def test_path_template_none():
+    path_template = "novars"
+    assert _parse_path_variables("one/two", path_template) is None