Added WiFiClient::setConnectTimeout

[DesktopMan]

Suggestion for connect timeout based on #73. No documentation yet, just looking for implementation feedback.

[DesktopMan]

I removed these comments as they seem to be completely unrelated. (Copy paste accident from earlier?)

[thekunalsaini]

it may be used later on

[DesktopMan]

But the comment was completely wrong. It was waiting to connect, not for the connection to close, and it waited 10 seconds not 4...

[thekunalsaini]

Okay

[DesktopMan]

Rebased for master. I didn't get any feedback so I'm asking if it can be merged now.

[fmatray]

Hello,
I'm very interested in this feature for a project.
I'll test it this week.

[fmatray]

I tried with a connexion IP:PORT without SSL. It work with with remote host up and, partially, when down.The call ServerDrv::startClient() seems to block for 8s, with remote host down.

My code test :

client->setConnectTimeout(1000);
Serial.println("Trying to connect");
now = millis();
connected = client->connect(IP, PORT);
Serial.print("End of try:");
Serial.println(millis() - now);

The Ethernet library have the same method but called setConnectionTimeout(). It would be nice to keep the same API.

I'm interested in this feature to make non blocking connexion.

[Alexander96f]

Hey are there any updates on this?
I have a similiar problem here: https://github.com/arduino-libraries/WiFiNINA/issues/144

[eugeniop]

Would be great to have this option in the library. Any reason why not merging it? @thekunalsaini ?

[lasselukkari]

I believe these changes will not actually work because the timeout value is not passed to the nina-fm. It still waits there for the default time.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[hisprofile]

what happened to this beautiful addition

[JAndrassy]

what happened to this beautiful addition

it doesn't work