Code review comment for lp:~vila/bzr/1509156-stricter-readonly

Richard Wilbur (richard-wilbur) wrote :

Looks like this is good for security! I approve.
+2

I am curious about the change in exception type raised by
bzrlib/transport/readonly.py: get_smart_client()
- NoSmartServer
+ NoSmartMedium

review: Approve

« Back to merge proposal