Fix integration to work with positional argument accepting middleware

Author: antonpirker

sentry_sdk/integrations/starlette.py

@@ -362,7 +362,7 @@ def patch_middlewares():
 
     if not_yet_patched:
         def _sentry_middleware_init(self, cls, *args, **kwargs):
-            # type: (Any, Any, Any) -> None
+            # type: (Any, Any, Any, Any) -> None
             if cls == SentryAsgiMiddleware:
                 return old_middleware_init(self, cls, *args, **kwargs)
 

tests/integrations/starlette/test_starlette.py

@@ -237,6 +237,10 @@ async def do_stuff(message):
 
         await self.app(scope, receive, do_stuff)
 
+class SampleMiddlewareWithArgs(Middleware):
+    def __init__(self, app, bla):
+        self.app = app
+        self.bla = bla
 
 class SampleReceiveSendMiddleware:
     def __init__(self, app):
@@ -862,6 +866,20 @@ def test_middleware_partial_receive_send(sentry_init, capture_events):
         idx += 1
 
 
+def test_middleware_args(sentry_init):
+    sentry_init(
+        traces_sample_rate=1.0,
+        integrations=[StarletteIntegration()],
+    )
+    _ = starlette_app_factory(
+        middleware=[Middleware(SampleMiddlewareWithArgs, "bla", "ba")]
+    )
+
+    # Only creating the App with an Middleware with args
+    # should not raise an error
+    # So as long as test passes, we are good
+
+
 def test_legacy_setup(
     sentry_init,
     capture_events,