[FLINK-29374] RpcConnectionTest may pass invalid rpc URL

• Type: Sub-task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.17.0
• Component/s: Runtime / RPC, Tests
• Labels: pull-request-available
• Assignee: Chesnay Schepler
• Reporter: Chesnay Schepler
• Created: 21/Sep/22 07:42
• Updated: 22/Sep/22 12:41
• Resolved: 22/Sep/22 12:41

Description

testConnectFailure() passed a URL under which no endpoint is running to check that this is handle correctly.
However the given URL may be completely invalid; the format of an RPC URL is an implementation detail of the RPC implementation, with the general contract being that you may only pass the return value of RpcSystem#getRpcUrl to connect() and not arbitrary values (and expect these to actually result in an actual connection attempt).

Issue Links

• links to: GitHub Pull Request #20868