[tbb-commits] [tor-browser-bundle-testsuite/master] Bug 40001: Add test for 'Communicating security expectations TB patch'

gk at torproject.org gk at torproject.org
Thu Oct 15 18:41:27 UTC 2020


commit a6c55026833dbdb0a7769d7352579fd796e7573c
Author: Alex Catarineu <acat at torproject.org>
Date:   Mon Jun 29 19:37:00 2020 +0200

    Bug 40001: Add test for 'Communicating security expectations TB patch'
---
 TBBTestSuite/TestSuite/BrowserBundleTests.pm       |   6 +
 marionette/setup.py                                |   1 +
 .../test_onion_security_expectations.py            | 134 +++++++++++++++++++++
 test-data/mixed.html                               |   8 ++
 test-data/mixed_active.html                        |   8 ++
 5 files changed, 157 insertions(+)

diff --git a/TBBTestSuite/TestSuite/BrowserBundleTests.pm b/TBBTestSuite/TestSuite/BrowserBundleTests.pm
index 4a8908e..834641e 100644
--- a/TBBTestSuite/TestSuite/BrowserBundleTests.pm
+++ b/TBBTestSuite/TestSuite/BrowserBundleTests.pm
@@ -476,6 +476,12 @@ our @tests = (
         # Bug 30432: temporarily disable test
         enable          => sub { undef },
     },
+    {
+        name            => 'onion_security_expectations',
+        type            => 'marionette',
+        use_net         => 1,
+        descr           => 'Check onion security expectations',
+    },
 );
 
 sub set_test_prefs {
diff --git a/marionette/setup.py b/marionette/setup.py
index 78f4522..9dc9772 100644
--- a/marionette/setup.py
+++ b/marionette/setup.py
@@ -5,6 +5,7 @@ PACKAGE_VERSION = '0.3'
 deps = [
         'marionette_harness == 5.0.0',
         'marionette_driver == 3.0.0',
+        'stem == 1.8.0',
 ]
 
 setup(name='tor-browser-tests',
diff --git a/marionette/tor_browser_tests/test_onion_security_expectations.py b/marionette/tor_browser_tests/test_onion_security_expectations.py
new file mode 100644
index 0000000..866ae1e
--- /dev/null
+++ b/marionette/tor_browser_tests/test_onion_security_expectations.py
@@ -0,0 +1,134 @@
+from marionette_driver import By, Wait
+from marionette_driver.errors import MarionetteException, NoSuchElementException, TimeoutException
+from marionette_driver.legacy_actions import Actions
+from marionette_harness import MarionetteTestCase, WindowManagerMixin
+
+import testsuite
+
+from stem.control import Controller
+
+from urlparse import urlparse
+
+import time
+
+class OnionFixturesMixin(object):
+    def setUp(self):
+        super(OnionFixturesMixin, self).setUp()
+        tor_control_port = testsuite.TestSuite(
+        ).t['options']['tor-control-port']
+        with Controller.from_port(port=int(tor_control_port)) as controller:
+            controller.authenticate()
+            port = urlparse(self.marionette.absolute_url('')).port
+            response = controller.create_ephemeral_hidden_service(
+                {80: port},
+                key_content='ED25519-V3',
+                await_publication=True,
+                detached=True,
+            )
+            self.service_id = response.service_id
+
+    def tearDown(self):
+        tor_control_port = testsuite.TestSuite(
+        ).t['options']['tor-control-port']
+        with Controller.from_port(port=int(tor_control_port)) as controller:
+            controller.authenticate()
+            controller.remove_ephemeral_hidden_service(self.service_id)
+        super(OnionFixturesMixin, self).tearDown()
+
+
+class Test(OnionFixturesMixin, WindowManagerMixin, MarionetteTestCase):
+    def get_identity_class(self):
+        with self.marionette.using_context('chrome'):
+            return self.marionette.execute_script('''
+                return document.getElementById("identity-box").className;
+            ''')
+
+    def get_identity_icon(self):
+        with self.marionette.using_context('chrome'):
+            return self.marionette.execute_script('''
+                const el = document.getElementById("identity-icon");
+                return document.defaultView.getComputedStyle(el)["list-style-image"];
+            ''')
+
+    def get_connection_type(self):
+        m = self.marionette
+        with self.marionette.using_context('chrome'):
+            self.execute_chrome(
+                'document.getElementById("identity-popup-more-info").click()')
+            m.switch_to_window(m.chrome_window_handles[1])
+            Wait(m, timeout=m.timeout.page_load).until(
+                lambda _: m.find_element('id', 'security-technical-shortform').text != '')
+            text = m.find_element('id', 'security-technical-shortform').text
+            m.close_chrome_window()
+            m.switch_to_window(self.start_window)
+            return text
+
+    def execute_chrome(self, script):
+        with self.marionette.using_context('chrome'):
+            return self.marionette.execute_script(script)
+
+    def test_onion_security_expectations(self):
+        m = self.marionette
+        m.timeout.implicit = 10
+
+        # Wait some more time to make sure the onion service is set up
+        time.sleep(10)
+
+        # Regular onion
+        with m.using_context('content'):
+            m.navigate('http://' + self.service_id + '.onion')
+            self.assertTrue(self.execute_chrome(
+                "return !!gIdentityHandler._isSecureConnection;"))
+            self.assertEqual(self.get_identity_class(), 'onionUnknownIdentity')
+            self.assertEqual(self.get_connection_type(),
+                             'Connection Encrypted (Onion Service)')
+            self.assertEqual(self.get_identity_icon(),
+                             'url("chrome://browser/skin/onion.svg")')
+
+        # Onion with mixed display content
+        with m.using_context('content'):
+            m.navigate('http://' + self.service_id + '.onion/mixed.html')
+            self.assertFalse(self.execute_chrome(
+                "return !!gIdentityHandler._isSecureConnection;"))
+            self.assertEqual(self.get_identity_class(),
+                             'onionUnknownIdentity onionMixedDisplayContent')
+            self.assertEqual(self.get_connection_type(),
+                             'Connection Partially Encrypted')
+            self.assertEqual(self.get_identity_icon(),
+                             'url("chrome://browser/skin/onion-warning.svg")')
+
+        # Onion with mixed active content
+        with m.using_context('content'):
+            m.navigate('http://' + self.service_id +
+                       '.onion/mixed_active.html')
+            self.assertTrue(self.execute_chrome(
+                "return !!gIdentityHandler._isSecureConnection;"))
+            self.assertEqual(self.get_identity_class(), 'onionUnknownIdentity')
+            self.assertEqual(self.get_connection_type(),
+                             'Connection Encrypted (Onion Service)')
+            self.assertEqual(self.get_identity_icon(),
+                             'url("chrome://browser/skin/onion.svg")')
+            # Reload with mixed content protection disabled
+            self.execute_chrome(
+                'gIdentityHandler.disableMixedContentProtection();')
+            Wait(m, timeout=m.timeout.page_load).until(
+                lambda _: self.get_identity_class() != 'onionUnknownIdentity')
+            self.assertFalse(self.execute_chrome(
+                "return !!gIdentityHandler._isSecureConnection;"))
+            self.assertEqual(self.get_identity_class(),
+                             'onionUnknownIdentity onionMixedActiveContent')
+            self.assertEqual(self.get_connection_type(),
+                             'Connection Partially Encrypted')
+            self.assertEqual(self.get_identity_icon(),
+                             'url("chrome://browser/skin/onion-slash.svg")')
+
+        # Onion with valid TLS certificate
+        with m.using_context('content'):
+            m.navigate('https://3g2upl4pq6kufc4m.onion/')
+            self.assertTrue(self.execute_chrome(
+                "return !!gIdentityHandler._isSecureConnection;"))
+            self.assertEqual(self.get_identity_class(), 'onionVerifiedDomain')
+            self.assertEqual(self.get_connection_type(
+            ), 'Connection Encrypted (Onion Service, TLS_AES_256_GCM_SHA384, 256 bit keys, TLS 1.3)')
+            self.assertEqual(self.get_identity_icon(),
+                             'url("chrome://browser/skin/onion.svg")')
diff --git a/test-data/mixed.html b/test-data/mixed.html
new file mode 100644
index 0000000..3de98a1
--- /dev/null
+++ b/test-data/mixed.html
@@ -0,0 +1,8 @@
+<!DOCTYPE html>
+<html>
+  <head>
+  </head>
+  <body>
+    <img src="http://www.torproject.org/static/images/tor-logo@2x.png"/>
+  </body>
+</html>
diff --git a/test-data/mixed_active.html b/test-data/mixed_active.html
new file mode 100644
index 0000000..23430ab
--- /dev/null
+++ b/test-data/mixed_active.html
@@ -0,0 +1,8 @@
+<!DOCTYPE html>
+<html>
+  <head>
+  </head>
+  <body>
+    <iframe src="http://www.torproject.org"></iframe>
+  </body>
+</html>





More information about the tbb-commits mailing list