Make it possible to time out TCP reads

[fhartwig]

This line is responsible for the stale cache issue in adserver, because sometimes it ends up waiting for data that never comes. This change adds a timeout option to the stream type which will be used to SetReadDeadline on the underlying TCP connection before calls to Read.

Question: Adding this as a parameter to the constructor function for Stream changes the API of this library. An alternative approach would be to leave the constructors as they are and add a setter method on Stream that the user would need to call on an existing Stream instance. I'm not sure how much we care about backwards compatibility here, so which way should I do this?

[fhartwig]

@pubnative/backend @bruwozniak Please have a look at this.

[alvelcom]

I'm not sure how much we care about backwards compatibility here, so which way should I do this?

It has 48 stars, but I guess it has no new users in the few past years. We're going to use standard mysql library anyway, let's not put a lot of effort into this one.

[kostyantyn]

An alternative approach would be to leave the constructors as they are and add a setter method on Stream that the user would need to call on an existing Stream instance

Unfortunately, in the current design, it won't be possible because of ConnectPlainHandshake function, which creates the stream and reads the first packets already. In this case, it won't be possible to respect the timeout for the initial read.

I think it's fine to extend the constructor as there are no promises to backward compatibility in this project yet. Actually would be good to denote in README that users must fix the commit of the package as new ones might be incompatible.