fix(wsgi): WSGI integrations respect SCRIPT_NAME env variable (#2622)

URLs generated using Sentry's WSGI Middleware should include SCRIPT_NAME in the event's url

Fixes #2576

---------

Co-authored-by: Daniel Szoke <[email protected]>

Author: sarvaSanjay

sentry_sdk/integrations/wsgi.py

@@ -55,10 +55,14 @@ def get_request_url(environ, use_x_forwarded_for=False):
     # type: (Dict[str, str], bool) -> str
     """Return the absolute URL without query string for the given WSGI
     environment."""
+    script_name = environ.get("SCRIPT_NAME", "").rstrip("/")
+    path_info = environ.get("PATH_INFO", "").lstrip("/")
+    path = f"{script_name}/{path_info}"
+
     return "%s://%s/%s" % (
         environ.get("wsgi.url_scheme"),
         get_host(environ, use_x_forwarded_for),
-        wsgi_decoding_dance(environ.get("PATH_INFO") or "").lstrip("/"),
+        wsgi_decoding_dance(path).lstrip("/"),
     )
 
 

tests/integrations/wsgi/test_wsgi.py

@@ -61,6 +61,25 @@ def test_basic(sentry_init, crashing_app, capture_events):
     }
 
 
+@pytest.mark.parametrize("path_info", ("bark/", "/bark/"))
+@pytest.mark.parametrize("script_name", ("woof/woof", "woof/woof/"))
+def test_script_name_is_respected(
+    sentry_init, crashing_app, capture_events, script_name, path_info
+):
+    sentry_init(send_default_pii=True)
+    app = SentryWsgiMiddleware(crashing_app)
+    client = Client(app)
+    events = capture_events()
+
+    with pytest.raises(ZeroDivisionError):
+        # setting url with PATH_INFO: bark/, HTTP_HOST: dogs.are.great and SCRIPT_NAME: woof/woof/
+        client.get(path_info, f"https://dogs.are.great/{script_name}")  # noqa: E231
+
+    (event,) = events
+
+    assert event["request"]["url"] == "https://dogs.are.great/woof/woof/bark/"
+
+
 @pytest.fixture(params=[0, None])
 def test_systemexit_zero_is_ignored(sentry_init, capture_events, request):
     zero_code = request.param