MikolajBalcerek commented on PR #45391: URL: https://github.com/apache/superset/pull/45391#issuecomment-6343970881
> Verified at [e9db698](https://github.com/apache/superset/commit/e9db698e67e72ff324fe1092bbf1eea04b52c644): the middleware now intercepts only the MCP endpoint path (trailing slash tolerated, mount-prefix safe via get_route_path), other routes pass through, and the new tests cover /mcp, /mcp/, HEAD, OAuth routes, custom mcp_path and a mounted prefix. My earlier NIT is fixed. CI had no failures when I approved; some jobs were still running. @aminghadersohi tests pass after rebasing on master, some unrelated issue to my PR. Merge ready! (I don't have merge rights) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
