Open vorot opened 4 years ago
Hi! I like most of the pull request. However, why don't you provide the local address as an optional parameter in the TftpClient constructor? That's also where the remote address is supplied, so it would kind of make sense, wouldn't it?
Hi! First of all, thank you for the library.
However, why don't you provide the local address as an optional parameter in the TftpClient constructor?
There are several reasons
TftpClient
assumes constructing the client object and creating the transfer object close to each other, which makes it not really important how to pass addresses.
Sometimes, UDP packets can be routed via a wrong (especially wireless) interface causing problems in TFTP protocol. In order to prevent that, an option to specify the network interface is added.