stacklok / toolhive / 21747048693

61%

Build Type

push

github

Committed by web-flow

Commit Message

Standardize proxy port field in MCPRemoteProxy (#3257)

Description:
This PR aligns MCPRemoteProxy with MCPServer by standardizing the proxy port field naming.

Summary
Added proxyPort to MCPRemoteProxySpec
Deprecated the legacy port field
Preserved backward compatibility during the transition
Details

GetProxyPort() now resolves ports in the following order:
1. proxyPort
2. port
3. default (8080)

Updated controller and deployment tests

Added test coverage to ensure proxyPort takes precedence over port

Verified existing resources using port continue to work

This improves API consistency across MCP resources while maintaining a smooth migration path for existing users.

Fixes #3102

---------

Co-authored-by: Don Browne <dmjb@users.noreply.github.com>

Run Details

9 of 12 new or added lines in 1 file covered. (75.0%)

4 existing lines in 2 files now uncovered.

41579 of 67981 relevant lines covered (61.16%)

75.84 hits per line