Add missing function getsocketoption

[rtpmsys]

WiFimanager.cpp connect function truncated all connnect timeout values below 1000ms to 0ms and rounded timeouts to the full second, e.g. 2.5 seconds became 2 seconds. For timeouts below 1000ms, the connect function failed. This fix handles timeouts correctly, by putting the fractional part of the timeout into the timeout_us variable.

Added function getsocketoptions, it was missing.

[me-no-dev]

This PR seems to include two non-relevant changes. Could you please split the timeout fix and getSocketOption into separate PRs?

[VojtechBartoska]

Any updates @rtpmsys?

[TD-er]

Why keep on insisting to have a WiFiClient::setTimeout function in this class using seconds while every other piece of code uses msec?
Cast it to Client and call that timeout will expect msec., which is a clear indication this design choice in the past was wrong.

[rtpmsys]

I submitted only the timeout correction and omitted the

This PR seems to include two non-relevant changes. Could you please split the timeout fix and getSocketOption into separate PRs?

I submitted just the timeout change (#7686) which fixes the error with connect timeout values below 1 second.

[rtpmsys]

Any updates @rtpmsys?

I submitted a new pull request (#7686) with just the timeout fix. Hope it makes it through the system, now.

[rtpmsys]

Why keep on insisting to have a WiFiClient::setTimeout function in this class using seconds while every other piece of code uses msec?
Cast it to Client and call that timeout will expect msec., which is a clear indication this design choice in the past was wrong.

You are right that the setTimeout function only expects seconds, I guess for compatibility reasons? The setTimeout luckily only seems to affect the write and read from the socket, not the connect.
My proposed fix (#7686) deals with the connect timeout, which comes before the write or read, therefore it is much more important that it can be made smaller than 1 second.

[mrengineer7777]

@rtpmsys #7686 was merged. Time to update this PR?

[rtpmsys]

Moved into another PR

[rtpmsys]

I will submit a new PR for adding the missing function getSocketOption.