Blacklist /support/ and /resources/ anywhere in the URL (#121); r=Ms2ger

manifest/tests/test_utils.py

@@ -6,18 +6,20 @@
 @pytest.mark.parametrize("url", [
     "/foo",
     "/tools/foo",
-    "/resources/foo",
     "/common/foo",
     "/conformance-checkers/foo",
-    "/_certs/foo"
+    "/_certs/foo",
+    "/resources/foo",
+    "/support/foo",
+    "/foo/resources/bar",
+    "/foo/support/bar"
 ])
 def test_is_blacklisted(url):
     assert is_blacklisted(url) is True
 
 
 @pytest.mark.parametrize("url", [
     "/foo/tools/bar",
-    "/foo/resources/bar",
     "/foo/common/bar",
     "/foo/conformance-checkers/bar",
     "/foo/_certs/bar"

manifest/utils.py

@@ -2,6 +2,7 @@
 from six import BytesIO
 
 blacklist = ["/tools/", "/resources/", "/common/", "/conformance-checkers/", "/_certs/"]
+blacklist_in = ["/resources/", "/support/"]
 
 def rel_path_to_url(rel_path, url_base="/"):
     assert not os.path.isabs(rel_path)
@@ -17,6 +18,9 @@ def is_blacklisted(url):
     for item in blacklist:
         if url.startswith(item):
             return True
+    for item in blacklist_in:
+        if item in url:
+            return True
     return False
 
 def from_os_path(path):