Updating tcp dynamic proxy to match on routes that are port only

fabiolb / fabio

Consul Load-Balancing made simple

https://fabiolb.net

MIT License

7.25k stars 621 forks source link

Closed nathanejohnson closed 3 years ago

nathanejohnson commented 3 years ago

This fixes #797

nathanejohnson commented 3 years ago

@murphymj25 could you take a look at this?

Thanks!

murphymj25 commented 3 years ago

@nathanejohnson It's been a while since I've looked at this code, but based tcp_proxy.go what you have looks correct.

killerdejavu commented 3 years ago

@nathanejohnson Would be great to have this PR merged! 😄

aaronhurt commented 3 years ago

LGTM, and thanks @murphymj25 for the confirmation.