Merge lp:~uriboni/webbrowser-app/basic-auth into lp:webbrowser-app

[Ugo Riboni]

Handle basic authentication requests by showing an authentication dialog

Depends on https://code.launchpad.net/~uriboni/oxide/basic-authentication/+merge/260922

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:999
http://jenkins.qa.ubuntu.com/job/webbrowser-app-ci/1860/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3116/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-amd64-ci/617/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-armhf-ci/617/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-i386-ci/617/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3114/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/webbrowser-app-ci/1860/rebuild

review: Needs Fixing (continuous-integration)

[lp:~uriboni/webbrowser-app/basic-auth]

1000. By Ugo Riboni on 2015-06-09

Add missing AP test file

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1000
http://jenkins.qa.ubuntu.com/job/webbrowser-app-ci/1862/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3120/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-amd64-ci/619/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-armhf-ci/619/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-i386-ci/619/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3118/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/webbrowser-app-ci/1862/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1000
http://jenkins.qa.ubuntu.com/job/webbrowser-app-ci/1869/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3156/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-amd64-ci/626/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-armhf-ci/626/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-i386-ci/626/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3154/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/webbrowser-app-ci/1869/rebuild

review: Needs Fixing (continuous-integration)

[Olivier Tilloy (osomon)]

263 + def tearDown(self):
264 + super(TestBasicAuthentication, self).tearDown()

This is useless.

[Olivier Tilloy (osomon)]

flake8 test fails.

review: Needs Fixing

[lp:~uriboni/webbrowser-app/basic-auth]

1001. By Ugo Riboni on 2015-06-17

Fix flake8 and clarify the purpose of some test code

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1001
http://jenkins.qa.ubuntu.com/job/webbrowser-app-ci/1901/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/3230/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-amd64-ci/658/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-armhf-ci/658/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/webbrowser-app-vivid-i386-ci/658/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/3228/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/webbrowser-app-ci/1901/rebuild

review: Needs Fixing (continuous-integration)

[Olivier Tilloy (osomon)]

Superseded by https://code.launchpad.net/~osomon/webbrowser-app/http-auth/+merge/263561.

review: Disapprove

Unmerged revisions

994. By Launchpad Translations on behalf of phablet-team on 2015-05-11

Launchpad automatic translations update.

993. By Launchpad Translations on behalf of phablet-team on 2015-05-10

Launchpad automatic translations update.

=== modified file 'debian/control'
--- debian/control	2015-05-27 05:31:12 +0000
+++ debian/control	2015-06-17 11:31:36 +0000
@@ -6,7 +6,7 @@
                debhelper (>= 9),
                dh-translations,
                hardening-wrapper,
-               liboxideqt-qmlplugin (>= 1.6),
+               liboxideqt-qmlplugin (>= 1.9),
                libqt5sql5-sqlite,
                python3-all,
                python3-flake8,
@@ -34,7 +34,7 @@
 Depends: ${misc:Depends},
          ${shlibs:Depends},
          fonts-liberation,
-         liboxideqt-qmlplugin (>= 1.7),
+         liboxideqt-qmlplugin (>= 1.9),
          libqt5sql5-sqlite,
          qml-module-qt-labs-folderlistmodel,
          qml-module-qt-labs-settings,
@@ -59,7 +59,7 @@
 Depends: ${misc:Depends},
          ${shlibs:Depends},
          fonts-liberation,
-         liboxideqt-qmlplugin (>= 1.5),
+         liboxideqt-qmlplugin (>= 1.9),
          libqt5sql5-sqlite,
          qml-module-qtquick2 | qtdeclarative5-qtquick2-plugin,
          qml-module-qtquick-window2 | qtdeclarative5-window-plugin,
@@ -96,7 +96,7 @@
 Pre-Depends: ${misc:Pre-Depends}
 Depends: ${misc:Depends},
          ${shlibs:Depends},
-         liboxideqt-qmlplugin (>= 1.6),
+         liboxideqt-qmlplugin (>= 1.9),
          qml-module-qtquick2 | qtdeclarative5-qtquick2-plugin,
          qml-module-qtquick-window2 | qtdeclarative5-window-plugin,
          qtdeclarative5-ubuntu-ui-toolkit-plugin,

=== modified file 'src/Ubuntu/Web/UbuntuWebView02.qml'
--- src/Ubuntu/Web/UbuntuWebView02.qml	2015-05-13 15:07:34 +0000
+++ src/Ubuntu/Web/UbuntuWebView02.qml	2015-06-17 11:31:36 +0000
@@ -18,7 +18,7 @@
 
 import QtQuick 2.0
 import QtQuick.Window 2.0
-import com.canonical.Oxide 1.5 as Oxide
+import com.canonical.Oxide 1.9 as Oxide
 import Ubuntu.Components 1.1
 import Ubuntu.Components.Popups 1.0
 import "." // QTBUG-34418

=== added file 'src/app/BasicAuthenticationDialog.qml'
--- src/app/BasicAuthenticationDialog.qml	1970-01-01 00:00:00 +0000
+++ src/app/BasicAuthenticationDialog.qml	2015-06-17 11:31:36 +0000
@@ -0,0 +1,76 @@
+/*
+ * Copyright 2015 Canonical Ltd.
+ *
+ * This file is part of webbrowser-app.
+ *
+ * webbrowser-app is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; version 3.
+ *
+ * webbrowser-app is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+import QtQuick 2.0
+import Ubuntu.Components 1.1
+import Ubuntu.Components.Popups 1.0 as Popups
+
+Popups.Dialog {
+    id: dialog
+    title: i18n.tr("Authentication required.")
+    // TRANSLATORS: %1 refers to the URL of the current website and %2 is a string that the website sends with more information about the authentication challenge (technically called "realm")
+    text: request ? i18n.tr('The website at %1 requires authentication. The website says "%2"').arg(request.host).arg(request.realm) : ""
+
+    property QtObject request: null
+
+    Connections {
+        target: request
+        onCancelled: PopupUtils.close(dialog)
+    }
+
+    TextField {
+        id: usernameInput
+        objectName: "username"
+        placeholderText: i18n.tr("Username")
+        onAccepted: {
+            request.allow(usernameInput.text, passwordInput.text)
+            PopupUtils.close(dialog)
+        }
+    }
+
+    TextField {
+        id: passwordInput
+        objectName: "password"
+        placeholderText: i18n.tr("Password")
+        echoMode: TextInput.Password
+        onAccepted: {
+            request.allow(usernameInput.text, passwordInput.text)
+            PopupUtils.close(dialog)
+        }
+    }
+
+    Button {
+        objectName: "allow"
+        text: i18n.tr("OK")
+        color: "green"
+        onClicked: {
+            request.allow(usernameInput.text, passwordInput.text)
+            PopupUtils.close(dialog)
+        }
+    }
+
+    Button {
+        objectName: "deny"
+        text: i18n.tr("Cancel")
+        color: UbuntuColors.coolGrey
+        onClicked: {
+            request.deny()
+            PopupUtils.close(dialog)
+        }
+    }
+}

=== modified file 'src/app/WebViewImpl.qml'
--- src/app/WebViewImpl.qml	2015-05-29 14:37:05 +0000
+++ src/app/WebViewImpl.qml	2015-06-17 11:31:36 +0000
@@ -66,6 +66,11 @@
         }
     }
 
+    onBasicAuthenticationRequested: {
+        PopupUtils.open(Qt.resolvedUrl("BasicAuthenticationDialog.qml"),
+                        webview.currentWebview, {"request": request})
+    }
+
     Loader {
         id: filePickerLoader
         source: formFactor == "desktop" ? "FilePickerDialog.qml" : "ContentPickerDialog.qml"

=== modified file 'tests/autopilot/webbrowser_app/emulators/browser.py'
--- tests/autopilot/webbrowser_app/emulators/browser.py	2015-05-28 13:56:52 +0000
+++ tests/autopilot/webbrowser_app/emulators/browser.py	2015-06-17 11:31:36 +0000
@@ -114,6 +114,9 @@
     def get_geolocation_dialog(self):
         return self.wait_select_single(GeolocationPermissionRequest)
 
+    def get_basic_auth_dialog(self):
+        return self.wait_select_single(BasicAuthenticationDialog)
+
     def get_selection(self):
         return self.wait_select_single(Selection)
 
@@ -255,6 +258,21 @@
         return self.select_single("Button", objectName="allow")
 
 
+class BasicAuthenticationDialog(uitk.UbuntuUIToolkitCustomProxyObjectBase):
+
+    def get_deny_button(self):
+        return self.select_single("Button", objectName="deny")
+
+    def get_allow_button(self):
+        return self.select_single("Button", objectName="allow")
+
+    def get_username_field(self):
+        return self.select_single("TextField", objectName="username")
+
+    def get_password_field(self):
+        return self.select_single("TextField", objectName="password")
+
+
 class Selection(uitk.UbuntuUIToolkitCustomProxyObjectBase):
 
     def get_rectangle(self):

=== modified file 'tests/autopilot/webbrowser_app/tests/http_server.py'
--- tests/autopilot/webbrowser_app/tests/http_server.py	2015-04-23 15:34:16 +0000
+++ tests/autopilot/webbrowser_app/tests/http_server.py	2015-06-17 11:31:36 +0000
@@ -6,6 +6,7 @@
 # under the terms of the GNU General Public License version 3, as published
 # by the Free Software Foundation.
 
+from base64 import b64decode
 import http.server as http
 import json
 import logging
@@ -31,6 +32,12 @@
         self.end_headers()
         self.wfile.write(html.encode())
 
+    def send_auth_request(self):
+        self.send_response(401)
+        self.send_header("WWW-Authenticate", "Basic realm=\"Enter Password\"")
+        self.end_headers()
+        self.send_html("Not Authorized")
+
     def do_GET(self):
         if self.path == "/ping":
             self.send_response(200)
@@ -140,6 +147,19 @@
             if query in self.suggestions_data:
                 suggestions = self.suggestions_data[query]
                 self.wfile.write(json.dumps(suggestions).encode())
+        elif self.path.startswith("/basic_auth"):
+            login = "user"
+            password = "pass"
+            if "Authorization" in self.headers:
+                header = self.headers.get("Authorization")
+                credentials = str(b64decode(header[len("Basic "):])).split(":")
+                if credentials[0] == login and credentials[1] == password:
+                    self.send_response(200)
+                    self.send_html("Authentication Successful !")
+                else:
+                    self.send_auth_request()
+            else:
+                self.send_auth_request()
         else:
             self.send_error(404)
 

=== added file 'tests/autopilot/webbrowser_app/tests/test_basic_authentication.py'
--- tests/autopilot/webbrowser_app/tests/test_basic_authentication.py	1970-01-01 00:00:00 +0000
+++ tests/autopilot/webbrowser_app/tests/test_basic_authentication.py	2015-06-17 11:31:36 +0000
@@ -0,0 +1,57 @@
+# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
+#
+# Copyright 2015 Canonical
+#
+# This program is free software: you can redistribute it and/or modify it
+# under the terms of the GNU General Public License version 3, as published
+# by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with this program. If not, see <http://www.gnu.org/licenses/>.
+
+from webbrowser_app.tests import StartOpenRemotePageTestCaseBase
+
+
+class TestBasicAuthentication(StartOpenRemotePageTestCaseBase):
+
+    def setUp(self):
+        super(TestBasicAuthentication, self).setUp()
+        self.username = "user"
+        self.password = "pass"
+        self.url = self.base_url + "/basic_auth"
+
+    def tearDown(self):
+        super(TestBasicAuthentication, self).tearDown()
+
+    def test_cancel(self):
+        self.main_window.go_to_url(self.url)
+        dialog = self.main_window.get_basic_auth_dialog()
+        self.pointing_device.click_object(dialog.get_deny_button())
+        dialog.wait_until_destroyed()
+
+    def test_right_credentials(self):
+        self.main_window.go_to_url(self.url)
+        dialog = self.main_window.get_basic_auth_dialog()
+        username = dialog.get_username_field()
+        username.write(self.username)
+        password = dialog.get_password_field()
+        password.write(self.password)
+        self.pointing_device.click_object(dialog.get_allow_button())
+        dialog.wait_until_destroyed()
+
+    def test_wrong_credentials(self):
+        self.main_window.go_to_url(self.url)
+        dialog = self.main_window.get_basic_auth_dialog()
+        username = dialog.get_username_field()
+        username.write("x")
+        password = dialog.get_password_field()
+        password.write("x")
+        self.pointing_device.click_object(dialog.get_allow_button())
+        dialog.wait_until_destroyed()
+        # verify that a new dialog has been displayed
+        self.main_window.get_basic_auth_dialog()