~bzr-pqm/bzr/bzr.dev

6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
1
# Copyright (C) 2011, 2012, 2013, 2016 Canonical Ltd
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
2
#
3
# This program is free software; you can redistribute it and/or modify
4
# it under the terms of the GNU General Public License as published by
5
# the Free Software Foundation; either version 2 of the License, or
6
# (at your option) any later version.
7
#
8
# This program is distributed in the hope that it will be useful,
9
# but WITHOUT ANY WARRANTY; without even the implied warranty of
10
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
11
# GNU General Public License for more details.
12
#
13
# You should have received a copy of the GNU General Public License
14
# along with this program; if not, write to the Free Software
15
# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
16
17
"""Tests for the SSL support in the urllib HTTP transport.
18
19
"""
20
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
21
import os
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
22
import ssl
23
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
24
from bzrlib import (
25
    config,
26
    trace,
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
27
)
6238.2.11 by Jelmer Vernooij
add basic tests for match_hostname.
28
from bzrlib.errors import (
29
    ConfigOptionValueError,
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
30
)
6238.2.11 by Jelmer Vernooij
add basic tests for match_hostname.
31
from bzrlib.tests import (
32
    TestCase,
33
    TestCaseInTempDir,
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
34
)
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
35
from bzrlib.transport.http import _urllib2_wrappers
36
37
38
class CaCertsConfigTests(TestCaseInTempDir):
39
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
40
    def get_stack(self, content):
41
        return config.MemoryStack(content.encode('utf-8'))
42
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
43
    def test_default_exists(self):
6437.25.1 by Vincent Ladeuil
Provide default paths for ca certs for supported platforms
44
        """Check that the default we provide exists for the tested platform."""
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
45
        stack = self.get_stack("")
6437.25.1 by Vincent Ladeuil
Provide default paths for ca certs for supported platforms
46
        self.assertPathExists(stack.get('ssl.ca_certs'))
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
47
48
    def test_specified(self):
49
        self.build_tree(['cacerts.pem'])
50
        path = os.path.join(self.test_dir, "cacerts.pem")
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
51
        stack = self.get_stack("ssl.ca_certs = %s\n" % path)
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
52
        self.assertEquals(path, stack.get('ssl.ca_certs'))
53
54
    def test_specified_doesnt_exist(self):
6437.25.6 by Vincent Ladeuil
Feedback from mgz.
55
        stack = self.get_stack('')
56
        # Disable the default value mechanism to force the behavior we want
57
        self.overrideAttr(_urllib2_wrappers.opt_ssl_ca_certs, 'default',
58
                          os.path.join(self.test_dir, u"nonexisting.pem"))
59
        self.warnings = []
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
60
6437.25.6 by Vincent Ladeuil
Feedback from mgz.
61
        def warning(*args):
62
            self.warnings.append(args[0] % args[1:])
63
        self.overrideAttr(trace, 'warning', warning)
64
        self.assertEquals(None, stack.get('ssl.ca_certs'))
65
        self.assertLength(1, self.warnings)
66
        self.assertContainsRe(self.warnings[0],
67
                              "is not valid for \"ssl.ca_certs\"")
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
68
69
70
class CertReqsConfigTests(TestCaseInTempDir):
71
72
    def test_default(self):
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
73
        stack = config.MemoryStack("")
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
74
        self.assertEquals(ssl.CERT_REQUIRED, stack.get("ssl.cert_reqs"))
75
76
    def test_from_string(self):
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
77
        stack = config.MemoryStack("ssl.cert_reqs = none\n")
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
78
        self.assertEquals(ssl.CERT_NONE, stack.get("ssl.cert_reqs"))
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
79
        stack = config.MemoryStack("ssl.cert_reqs = required\n")
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
80
        self.assertEquals(ssl.CERT_REQUIRED, stack.get("ssl.cert_reqs"))
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
81
        stack = config.MemoryStack("ssl.cert_reqs = invalid\n")
6238.2.10 by Jelmer Vernooij
Add more tests for ssl.ca_certs option.
82
        self.assertRaises(ConfigOptionValueError, stack.get, "ssl.cert_reqs")
6238.2.11 by Jelmer Vernooij
add basic tests for match_hostname.
83
84
85
class MatchHostnameTests(TestCase):
86
87
    def test_no_certificate(self):
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
88
        self.assertRaises(ValueError,
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
89
                          ssl.match_hostname, {}, "example.com")
6238.2.11 by Jelmer Vernooij
add basic tests for match_hostname.
90
6573.1.1 by Andrew Starr-Bochicchio
Fix possible abuse of _urllib2_wrappers.match_hostname() for denial of service using certificates with many wildcards (CVE-2013-2099).
91
    def test_wildcards_in_cert(self):
92
        def ok(cert, hostname):
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
93
            ssl.match_hostname(cert, hostname)
94
95
        def not_ok(cert, hostname):
96
            self.assertRaises(
97
                ssl.CertificateError,
98
                ssl.match_hostname, cert, hostname)
6573.1.1 by Andrew Starr-Bochicchio
Fix possible abuse of _urllib2_wrappers.match_hostname() for denial of service using certificates with many wildcards (CVE-2013-2099).
99
100
        # Python Issue #17980: avoid denials of service by refusing more than
101
        # one wildcard per fragment.
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
102
        ok({'subject': ((('commonName', 'a*b.com'),),)}, 'axxb.com')
103
        not_ok({'subject': ((('commonName', 'a*b.co*'),),)}, 'axxb.com')
104
        not_ok({'subject': ((('commonName', 'a*b*.com'),),)}, 'axxbxxc.com')
6573.1.1 by Andrew Starr-Bochicchio
Fix possible abuse of _urllib2_wrappers.match_hostname() for denial of service using certificates with many wildcards (CVE-2013-2099).
105
6238.2.11 by Jelmer Vernooij
add basic tests for match_hostname.
106
    def test_no_valid_attributes(self):
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
107
        self.assertRaises(ssl.CertificateError, ssl.match_hostname,
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
108
                          {"Problem": "Solved"}, "example.com")
6238.2.11 by Jelmer Vernooij
add basic tests for match_hostname.
109
110
    def test_common_name(self):
111
        cert = {'subject': ((('commonName', 'example.com'),),)}
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
112
        self.assertIs(None,
6613.1.1 by Vincent Ladeuil
Use ssl module for the match_hostname function
113
                      ssl.match_hostname(cert, "example.com"))
114
        self.assertRaises(ssl.CertificateError, ssl.match_hostname,
6238.2.19 by Vincent Ladeuil
Just hack the tests until they pass.
115
                          cert, "example.org")