Here is the summery of the patch: * connman should conduct WPS protocol without specifying SSID/band. * connman should save a received credential into applicable wifi service after completing WPS protocol. * Then connman should notify WPS success with credential saved service list. * After that, user selects and calls connect from service list. * User also can retry the connection without retrying WPS even if the connection to AP gets unstable. Previous connman wps API conducts WPS procedure with specific wifi service (SSID), which causes following issues in those cases: * Multi-band Operation: - Many APs in the market operate WPS protocol over multi-band and the SSID is different between band A and band B. - However, current connman WPS implementation cannot specify the operation band. - Therefore, wpa_supplicant might proceed with conducting WPS procedure and connecting to SSID (AP) which is different from connman-specified SSID (service). - In such case, connman disconnects WPS connection and judges as a failure. * Multi credential delivery: - There might be a case the AP distributes multiple credentials. - In current implementation, the wpa_supplicant might select and connect with the SSID which is different from specified service, too. - connman disconnects this WPS connection and judges as a failure * Unconfigured AP: - The SSID and security setting would be updated after the WPS procedure. - wpa_supplicant can connect with correct AP, but the SSID and setting are different from specified service then connman disconnects with AP. So, user would see such problems and since those scenarios are tested during WPS certification testing, current connman cannot pass the WPS certification. Moreover, connman discards WPS credential when connman fails the association with AP even after WPS protocol has successfully completed. In this case, the user need retry WPS protocol, so I think WPS protocol and connection should be separated. Signed-off-by: n-itaya <Natsuki.Itaya(a)sony.com&gt; --- doc/agent-api.txt | 6 ++++++ doc/technology-api.txt | 43 ++++++++++++++++++++++++++++++++++++++++++ 2 files changed, 49 insertions(+) diff --git a/doc/agent-api.txt b/doc/agent-api.txt index e3c1dcde..f52ed4bd 100644 --- a/doc/agent-api.txt +++ b/doc/agent-api.txt @@ -141,6 +141,12 @@ Fields string Name In case of a RequestPeerAuthorization, this field will be set as mandatory. + To use this in order to get connected to a given + services is deprecated and should not be used. Methods + Technology.StartStaWps and Technology.StartApWps + are provided by ConnMan and user should use those two + instead. + string Username Username for WISPr authentication. This field will be diff --git a/doc/technology-api.txt b/doc/technology-api.txt index f22e9b29..62d2c612 100644 --- a/doc/technology-api.txt +++ b/doc/technology-api.txt @@ -40,6 +40,49 @@ Methods dict GetProperties() [deprecated] via the PeersChanged signal from the manager interface. + void StartApWps(string authentication) + + Start a WPS Session when the system is playing AP + role (Tethering) in order to allow potential Ex-STAs + to get connected by using WPS. + + The argument indicates the WPS authentication method + which can be an empty string, if user wants to use + push-button method, or a pin code if user wants to + use the pin method. + + This method is supported only by WiFi technology. + + Possible Errors: [service].Error.InvalidArguments + [service].Error.PermissionDenied + [service].Error.NotSupported + + void StartStaWps(string authentication) + + Start a WPS Session in STA role in order to be able + to get connected with an Ex-AP with WPS capabilities. + + The argument indicates the WPS authentication method + which can be an empty string, if user wants to use + push-button method, or a pin code if user wants to + use the pin method. + + This method is supported only by WiFi technology. + + Possible Errors: [service].Error.InvalidArguments + [service].Error.OperationAborted + [service].Error.OperationTimeout + [service].Error.NotConnected + [service].Error.NotSupported + + void CancelWps() + + Cancel ongoing WPS session. + + This method is supported only by WiFi technology. + + Possible Errors: [service].Error.NotSupported + Signals PropertyChanged(string name, variant value) This signal indicates a changed value of the given -- 2.17.1