hostapd: add fallback for WPS on stations
authorRaphaël Mélotte <raphael.melotte@mind.be>
Fri, 29 Oct 2021 08:45:06 +0000 (10:45 +0200)
committerDaniel Golle <daniel@makrotopia.org>
Mon, 27 Dec 2021 16:32:02 +0000 (16:32 +0000)
commit69ce75fb12526e11f79618a7cbedb32335b1f2bc
tree8cb549b119c201246ae16bafa6783de689657b76
parentc555c34dd15c11a901ea4dfcadbcc0e4f3072160
hostapd: add fallback for WPS on stations

Up to now the WPS script triggered WPS on the stations only if it
could not trigger it successfully on any hostapd instance.

In a Multi-AP context, there can be a need (to establish a new
wireless backhaul link) to trigger WPS on the stations, regardless of
whether there is already a hostapd instance configured or not. The
current script makes it impossible, as if hostapd is running and
configured, WPS would always be triggered on hostapd only.

To allow both possibilities, the following changes are made:

- Change the "pressed" action to "release", so that we can make use of
the "$SEEN" variables (to know for how long the button was pressed).

- If the button is pressed for less than 3 seconds, keep the original
behavior.

- If the button is pressed for 3 seconds or more, trigger WPS on the
stations, regardless of the status of any running hostapd instance.

- Add comments explaining both behaviors.

- While at it, replace the usage of '-a' with a '[] && []'
construct (see [1]).

This gives users a "fallback" mechanism to onboard a device to a
Multi-AP network, even if the device already has a configured hostapd
instance running.

[1]: https://github.com/koalaman/shellcheck/wiki/SC2166

Signed-off-by: Raphaël Mélotte <raphael.melotte@mind.be>
package/network/services/hostapd/files/wps-hotplug.sh