Skip to content

client-routes: include original port in endpoint identity#880

Closed
dkropachev wants to merge 1 commit intomasterfrom
dk/client-routes-endpoint
Closed

client-routes: include original port in endpoint identity#880
dkropachev wants to merge 1 commit intomasterfrom
dk/client-routes-endpoint

Conversation

@dkropachev
Copy link
Copy Markdown
Collaborator

Summary

  • include original port in ClientRoutesEndPoint equality, hash, ordering, and repr
  • add unit coverage for port-sensitive identity and ordering with missing original ports

Verification

  • repo_ci fast reached workflow-test passed and then timed out at the 300s wrapper limit

@dkropachev dkropachev requested a review from sylwiaszunejko May 7, 2026 23:11
@dkropachev
Copy link
Copy Markdown
Collaborator Author

Actuslly it is a bad idea to include port into it,

@dkropachev dkropachev closed this May 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant