Changes between Version 1 and Version 2 of Ticket #3064, comment 1


Ignore:
Timestamp:
06/15/2017 11:43:09 AM (7 years ago)
Author:
Cas
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #3064, comment 1

    v1 v2  
    11Hmm ok, trying solve that issue has unintentionally broken server chaining. I'd want to avoid pem module in 1.3-stable as it pushes the minimum Twisted version to 14.0 but may consider it for develop. 
    22 
    3 Can you test this change: 
    4  
    5 {{{#!diff 
    6 diff --git deluge/ui/web/server.py deluge/ui/web/server.py 
    7 index 2897ae323..32df20ddf 100644 
    8 --- deluge/ui/web/server.py 
    9 +++ deluge/ui/web/server.py 
    10 @@ -661,7 +661,9 @@ def start_ssl(self): 
    11          with open(configmanager.get_config_dir(self.pkey)) as pkey: 
    12              private_key = KeyPair.load(pkey.read(), FILETYPE_PEM).original 
    13          options = CertificateOptions(privateKey=private_key, certificate=certificate, method=SSL.SSLv23_METHOD) 
    14 -        options.getContext().set_options(SSL.OP_NO_SSLv2 | SSL.OP_NO_SSLv3) 
    15 +        ctx = options.getContext() 
    16 +        ctx.set_options(SSL.OP_NO_SSLv2 | SSL.OP_NO_SSLv3) 
    17 +        ctx.use_certificate_chain_file(configmanager.get_config_dir(self.cert)) 
    18   
    19          self.socket = reactor.listenSSL(self.port, self.site, options, interface=self.interface) 
    20          ip = self.socket.getHost().host 
    21 }}} 
     3Can you test this change in PR: https://github.com/deluge-torrent/deluge/pull/161