wispr: Set LoginRequired when wispr login is requested

Setting it while handling redirection is not bullet proof in case
of AP which does not implement well wispr and sends wispr content
with no redirection at first place.

Fixes BMC#23424

diff --git a/src/wispr.c b/src/wispr.c
index 1f470f7..7cc2b99 100644 (file)
--- a/src/wispr.c
+++ b/src/wispr.c
@@ -482,6 +482,8 @@ static gboolean wispr_manage_message(GWebResult *result,
 
                wp_context->wispr_result = CONNMAN_WISPR_RESULT_LOGIN;
 
+               __connman_service_request_login(wp_context->service);
+
                if (__connman_agent_request_login_input(wp_context->service,
                                        wispr_portal_request_wispr_login,
                                        wp_context) != -EIO)
@@ -570,8 +572,6 @@ static gboolean wispr_portal_web_result(GWebResult *result, gpointer user_data)
 
                DBG("Redirect URL: %s", redirect);
 
-               __connman_service_request_login(wp_context->service);
-
                wp_context->request_id = g_web_request_get(wp_context->web,
                                redirect, wispr_portal_web_result, wp_context);