Code review comment for lp:~vila/bzr/webdav-in-core

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

I've had a closer look at the code now. It looks fairly reasonable, though there aren't a lot of Transport-level tests (is that done by bzrlib.tests.per_transport ?).

It's a bit surprising to see _raise_curl_http_error despite the backend being based on the urllib HTTP implementation. Is that a bug?

review: Needs Information

« Back to merge proposal