superfly / flyctl

Command line tools for fly.io services
https://fly.io
Apache License 2.0
1.36k stars 224 forks source link

Display proxy ephemeral port when the user specifies 0 #3639

Closed jonatanklosko closed 3 weeks ago

jonatanklosko commented 3 weeks ago

The current behaviour is to print 0, which is not meaningful:

$ fly proxy 0:5558 ...
Proxying local port 0 to remote [...]:5558

With this change we special case port 0 and print the actual ephemeral port assigned to the listener.

dangra commented 3 weeks ago

thanks @jonatanklosko