Switch default port to something other than `80`
mellonnen opened this issue · 2 comments
mellonnen commented
ZinoKader commented
Yes, the default should be some ephemeral TCP port, not 80 which is HTTP.
How about 65432?
mellonnen commented
Also, it does not really make sense for the sender/receiver to have to provide IP and port individually. In the case, you want to send through a self-hosted rendezvous server you would have to provide both. We should probably accommodate both the Sender
and Receiver
stucts to accept an option for IP and port , and join the port and address cli flags into one flag.
edit: we could create an option
WithRendevouzTCPAddr(add net.TCPAddr)
for the Sender
and Receiver
structs, which logically makes more sense then providing port/address in two different options