Fix request.url with full URI as SERVER_NAME

sanic/request.py

@@ -619,8 +619,8 @@ def scheme(self) -> str:
         :return: http|https|ws|wss or arbitrary value given by the headers.
         :rtype: str
         """
-        if "//" in self.app.config.get("SERVER_NAME", ""):
-            return self.app.config.SERVER_NAME.split("//")[0]
+        if "://" in self.app.config.get("SERVER_NAME", ""):
+            return self.app.config.SERVER_NAME.split("://")[0]
         if "proto" in self.forwarded:
             return str(self.forwarded["proto"])
 

tests/test_requests.py

@@ -1917,16 +1917,26 @@ def handler(request):
         return text("ok")
 
     app.config.SERVER_NAME = "my-server"  # This means default port
-    assert app.url_for("handler", _external=True) == "http://my-server/foo"
+    path = "/foo"
+    url = f"http://my-server{path}"
+    assert app.url_for("handler", _external=True) == url
     request, response = app.test_client.get("/foo")
-    assert request.url_for("handler") == f"http://my-server/foo"
+    assert request.url_for("handler") == url
+    assert request.scheme == "http"
+    assert request.path == path
+    assert request.url == url
 
     app.config.SERVER_NAME = "https://my-server/path"
     request, response = app.test_client.get("/foo")
-    url = f"https://my-server/path/foo"
+    path = "/path/foo"
+    url = f"https://my-server{path}"
     assert app.url_for("handler", _external=True) == url
     assert request.url_for("handler") == url
-
+    assert request.scheme == "https"
+    # These two currently fail, because SERVER_NAME’s path is not
+    # taken into account.
+    #assert request.path == path
+    #assert request.url == url
 
 def test_url_for_with_forwarded_request(app):
     @app.get("/")