fix(arthas): use declared operations for UI calls#20
Merged
Conversation
The Arthas UI still used the old arbitrary HTTP handler model for console, MCP, and direct /api proxy requests. Those paths are no longer mounted by the plugin SDK and can be blocked or misrouted by the host proxy.\n\nRoute command execution through the declared exec operation, remove the dead HTTPHandler proxy/MCP code, stop starting MCP port-forwards, and drop UI tabs/links that depended on arbitrary plugin endpoints.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The Arthas UI still depended on the old arbitrary plugin HTTP handler model for console, MCP, and direct /api proxy calls. Those paths are no longer mounted by the SDK and can be blocked or misrouted by the host plugin proxy.\n\nThis routes command execution through the declared exec operation, removes the dead HTTPHandler proxy/MCP code, stops starting MCP port-forwards, and drops UI entry points that depended on arbitrary plugin endpoints.