Merge pull request #18509 from mvdbeek/fix_statsd_client

[24.1] Apply statsd arg sanitization to all pages

lib/galaxy/web/framework/base.py

@@ -251,8 +251,13 @@ def handle_request(self, request_id, path_info, environ, start_response, body_re
                 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()