serve: get params and fragments should be ignored (#80)

fixes #79

exhibition/tests/test_serve.py

@@ -109,6 +109,30 @@ def test_fetch_file(self):
         headers = dict(response.getheaders())
         self.assertEqual(headers["Cache-Control"], "no-store")
 
+    def test_fetch_file_with_get_params(self):
+        settings = Config({"deploy_path": self.tmp_dir.name, "content_path": self.tmp_dir.name})
+        self.get_server(settings)
+
+        self.client.request("GET", "/style.css?something=1")
+        response = self.client.getresponse()
+        self.assertEqual(response.status, 200)
+        content = response.read()
+        self.assertEqual(content, CSS_CONTENTS.encode())
+        headers = dict(response.getheaders())
+        self.assertEqual(headers["Cache-Control"], "no-store")
+
+    def test_fetch_file_with_fragment(self):
+        settings = Config({"deploy_path": self.tmp_dir.name, "content_path": self.tmp_dir.name})
+        self.get_server(settings)
+
+        self.client.request("GET", "/style.css#something")
+        response = self.client.getresponse()
+        self.assertEqual(response.status, 200)
+        content = response.read()
+        self.assertEqual(content, CSS_CONTENTS.encode())
+        headers = dict(response.getheaders())
+        self.assertEqual(headers["Cache-Control"], "no-store")
+
     def test_fetch_file_with_prefix(self):
         settings = Config({"deploy_path": self.tmp_dir.name, "content_path": self.tmp_dir.name,
                            "base_url": "/bob/"})

exhibition/utils.py

@@ -47,6 +47,8 @@ def gen(settings):
 class ExhibitionBaseHTTPRequestHandler(SimpleHTTPRequestHandler):
     def _sanitise_path(self, path):
         """ Strip leading and trailing / as well as base_url, if preset """
+        path = path.split('?', 1)[0]
+        path = path.split('#', 1)[0]
         path = path.strip("/")
         if self._settings.get("base_url"):
             base = self._settings["base_url"].strip("/")