Merge lp:~andidog/bzr-webdav/trunk into lp:bzr-webdav

[Andreas Sommer]

Fix for change in bzrlib's SSL certificate checking: Parameter ca_certs was added to the Opener class. I added this as keyword argument with default value None, so that the webdav plugin should still work with older Bazaar versions.

See bzr revision "6238.2.22 revid:<email address hidden>" for the change.

[Andreas Sommer (andidog)]

Sorry, in fact it won't work with older versions because I'm also passing that parameter on to the super class.

[Vincent Ladeuil (vila)]

Thanks for looking into it, but yes, this is most likely unrelated to your push issue. The ca_certs parameter is only relevant for tests (for now).

[Andreas Sommer (andidog)]

I didn't say this is related to my push issue. Actually I first found out that pushing leads to an exception because of the missing ca_certs parameter (new in the revision I mentioned above). What I posted in the mailing list is something completely different.

I added the ca_certs parameter to __init__ here because the DavOpener class gets assigned to HttpDavTransport._opener_class, which in turn is used as follows (note the construction uses the ca_certs parameter):

class HttpTransport_urllib(http.HttpTransportBase):
    """Python urllib transport for http and https."""

    # In order to debug we have to issue our traces in sync with
    # httplib, which use print :(
    _debuglevel = 0

    _opener_class = Opener

    def __init__(self, base, _from_transport=None, ca_certs=None):
        super(HttpTransport_urllib, self).__init__(
            base, 'urllib', _from_transport=_from_transport)
        if _from_transport is not None:
            self._opener = _from_transport._opener
        else:
            self._opener = self._opener_class(
                report_activity=self._report_activity, ca_certs=ca_certs)

[Vincent Ladeuil (vila)]

Ooops, damn, will look into this asap :-/

Thanks for the heads-up !

[Vincent Ladeuil (vila)]

Sorry for the delay, lp:bzr-webdav now contains a moral equivalent of this proposal, thanks again for the heads-up.

review: Approve

=== modified file 'webdav.py'
--- webdav.py	2011-11-18 10:43:21 +0000
+++ webdav.py	2012-03-07 11:57:18 +0000
@@ -387,9 +387,10 @@
 class DavOpener(_urllib2_wrappers.Opener):
     """Dav specific needs regarding HTTP(S)"""
 
-    def __init__(self, report_activity=None):
+    def __init__(self, report_activity=None, ca_certs=None):
         super(DavOpener, self).__init__(connection=DavConnectionHandler,
-                                        report_activity=report_activity)
+                                        report_activity=report_activity,
+                                        ca_certs=ca_certs)
 
 
 class HttpDavTransport(_urllib.HttpTransport_urllib):