Unverified Commit cbbb5ef4 authored by Dannon's avatar Dannon Committed by GitHub
Browse files

Merge pull request #18509 from mvdbeek/fix_statsd_client

[24.1] Apply statsd arg sanitization to all pages
parents 20cc846b 6be9f235
Loading
Loading
Loading
Loading
+6 −1
Original line number Diff line number Diff line
@@ -251,8 +251,13 @@ class WebApplication:
                raise
        trans.controller = controller_name
        trans.action = action

        # Action can still refer to invalid and/or inaccurate paths here, so we use the actual
        # controller and method names to set the timing key.

        action_tag = getattr(method, "__name__", "default")
        environ["controller_action_key"] = (
            f"{'api' if environ['is_api_request'] else 'web'}.{controller_name}.{action or 'default'}"
            f"{'api' if environ['is_api_request'] else 'web'}.{controller_name}.{action_tag}"
        )
        # Combine mapper args and query string / form args and call
        kwargs = trans.request.params.mixed()