Merge pull request #289 from adrienthiery/fix/hashtag-url

fix: url validator considers urls with /#/ as valid

Author: yozachar

src/validators/url.py

@@ -116,6 +116,7 @@ def _validate_optionals(path: str, query: str, fragment: str):
     if query:
         optional_segments &= bool(_query_regex().match(query))
     if fragment:
+        fragment = fragment.lstrip("/") if fragment.startswith("/") else fragment
         optional_segments &= all(char_to_avoid not in fragment for char_to_avoid in ("/", "?"))
     return optional_segments
 

tests/test_url.py

@@ -84,6 +84,7 @@
         "https://travel-usa.com/wisconsin/旅行/",
         "http://:::::::::::::@exmp.com",
         "http://-.~_!$&'()*+,;=:%40:80%2f::::::@example.com",
+        "https://exchange.jetswap.finance/#/swap",
         # when simple_host=True
         # "http://localhost",
         # "http://localhost:8000",