Merge lp:~spiv/bzr/escape-smart-server-requested-paths-458762 into lp:bzr
Proposed by
Andrew Bennetts
Status: | Merged |
---|---|
Merged at revision: | not available |
Proposed branch: | lp:~spiv/bzr/escape-smart-server-requested-paths-458762 |
Merge into: | lp:bzr |
Diff against target: |
134 lines 6 files modified
NEWS (+3/-0) bzrlib/smart/request.py (+1/-1) bzrlib/smart/vfs.py (+9/-0) bzrlib/tests/test_smart.py (+26/-0) bzrlib/tests/test_smart_transport.py (+3/-1) doc/developers/network-protocol.txt (+3/-0) |
To merge this branch: | bzr merge lp:~spiv/bzr/escape-smart-server-requested-paths-458762 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Martin Pool | Approve | ||
Review via email: mp+14126@code.launchpad.net |
To post a comment you must log in.
This fixes inconsistencies in server-side HPSS path decoding that was breaking unicode paths. The client has always been consistent in how it was transmitting requests (VFS requests use URL escaped paths, others use UTF-8), but the server was assuming URL escaped everywhere. This branch fixes the server to match reality, and adds some tests (and fixes an existing one!). Thanks very much to Michael Hudson for investigating this issue and writing the first version of this patch.