Check repeatedly for matching SSID on iOS

[elliottkember]

I've found locally that sometimes the completionHandler for [NEHotspotConfigurationManager sharedManager] applyConfiguration returns before the network is fully connected.

There is an Apple Support thread about this here: https://developer.apple.com/forums/thread/111379 -

NEHotspotConfigurationManager calls its completion handler as soon as it has dealt with the request, that is, as soon as it has checked the request and passed it on to that Wi-Fi infrastructure. This has a significant impact on users of NEHotspotConfigurationManager: You can’t be guaranteed that the device has actually joined the Wi-Fi network by the time your completion handler is called. In fact this won’t be the case most of the time.
If there’s an error joining the Wi-Fi network, that error is reported to the user, not via your completion handler.

This patch checks the connected SSID every 0.5 seconds for 10 seconds, to see whether it's connected. I've been testing this locally and it seems to work great, so thought I should submit it for review in case this is helpful or I'm doing something wrong.

[elliottkember]

Bumping this pull request, because we've been using it locally with great results ever since. This is a useful addition for connecting to small devices like 360 cameras (that's our use case).

[JuanSeBestia]

At the moment I can't test it, if some other mantainers like @eliaslecomte or @gitisaac want to test, approve and merge it, go ahead

[alexma01]

Hi guys,

can we merge this PR?

Thanks

[JuanSeBestia]

@alexma01 are you already test it?

[JuanSeBestia]

🎉 This PR is included in version 4.11.0 🎉

The release is available on:

• GitHub release
• npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

[bitcrumb]

@JuanSeBestia @elliottkember Thank you for this! This has been on my to do list for way too long. Much appreciated!

[elliottkember]

@bitcrumb A pleasure. Very happy to see this merged. It was nice to delete the patch we've been using. We haven't had any problems since this patch, so hoping others see the same great results.