fix: properly set cancel socket timeout

[Powerrr]

If we set a socket timeout on PGStream's socket, but don't set PGStream.pgInput.timeoutRequested to true, then PGStream.pgInput will ignore SocketTimeoutExceptions (VisibleBufferedInputStream.java#L169) and the timeout effectively doesn't work.

PGStream.setNetworkTimeout() sets both the timeout and the timeoutRequested field.

All Submissions:

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Changes to Existing Features:

• Does this break existing behaviour? If so please explain.
• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your core changes, as applicable?
• Have you successfully run tests with your changes locally?

[davecramer]

Is there a way to test this ? Seems it wasn't working before and we didn't catch it. I'd like to see a test added to this to ensure that we don't break it in the future

[Powerrr]

Yes, we can test this. Added a test to check that cancel() doesn't hang forever.

[Powerrr]

Any news?

[vlsi]

This PR looks good to me.
timeoutRequested is indeed surprising