Merge b82d982 into 0a8b83c

baseframe/filters.py

@@ -10,6 +10,7 @@
 from flask import Markup, request
 
 from babel.dates import format_date, format_datetime, format_time
+from furl import furl
 from pytz import UTC
 
 from coaster.gfm import markdown
@@ -287,6 +288,15 @@ def timestamp_filter(value):
     return ts
 
 
+@baseframe.app_template_filter('cleanurl')
+def cleanurl_filter(url):
+    if not isinstance(url, furl):
+        url = furl(url)
+    url.path.normalize()
+    netloc = url.netloc.lstrip('www.') if url.netloc else url.netloc
+    return furl().set(netloc=netloc, path=url.path).url.lstrip('//').rstrip('/')
+
+
 @baseframe.app_template_filter('make_relative_url')
 def make_relative_url(url):
     """Filter to discard scheme and netloc from a URL, used to undo _external=True"""

tests/test_filters.py

@@ -363,3 +363,25 @@ def test_none_if_empty(self):
         assert none_if_empty_func([]) is None
         assert none_if_empty_func(False) is None
         assert none_if_empty_func(0) is None
+
+    def test_cleanurl(self):
+        assert (
+            filters.cleanurl_filter("https://www.example.com/some/path/?query=value")
+            == "example.com/some/path"
+        )
+        assert (
+            filters.cleanurl_filter("//example.com/some/path/?query=value")
+            == "example.com/some/path"
+        )
+        assert filters.cleanurl_filter("http://www.example.com/") == "example.com"
+        assert filters.cleanurl_filter("//www.example.com/") == "example.com"
+        assert filters.cleanurl_filter("//test/") == "test"
+        # cannot process if scheme is missing and no // to begin with
+        assert (
+            filters.cleanurl_filter("www.example.com/some/path/")
+            == "www.example.com/some/path"
+        )
+        assert (
+            filters.cleanurl_filter("example.com/some/path") == "example.com/some/path"
+        )
+        assert filters.cleanurl_filter("foobar") == "foobar"