Fixes T11079. We currently un-escape / in phutil_escape_uri() because it is safe to leave unescaped in paths and an awful UX to escape, but it must be escaped in usernames/passwords.
Details
Details
- Added failing unit test.
- Fixed test.
- Created a credential with username and password containing a slash.
- Set this credential on an observation URI for a repository.
- Used bin/repository update R... to pull the repostiory.
- Before patch, got an error about a malfored URI.
- After patch, things worked properly.
Cloning into bare repository '/Users/epriestley/dev/core/repo/local/55'... fatal: unable to access 'https://slash/slash:slash/slash@github.com/epriestley/poems.git/': Could not resolve host: slash
Diff Detail
Diff Detail
- Repository
- rPHU libphutil
- Branch
- uri1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 12405 Build 15706: Run Core Tests Build 15705: arc lint + arc unit