Page MenuHomePhabricator

Make PhutilGitURI and PhutilURI strict about leading whitespace
ClosedPublic

Authored by epriestley on Apr 29 2014, 5:34 PM.
Tags
None
Referenced Files
F14008199: D8897.id21121.diff
Tue, Oct 29, 5:22 PM
F13981951: D8897.id.diff
Sat, Oct 19, 8:12 PM
F13981731: D8897.id21121.diff
Sat, Oct 19, 6:41 PM
F13981625: D8897.diff
Sat, Oct 19, 5:57 PM
F13974681: D8897.diff
Fri, Oct 18, 6:49 AM
F13969963: D8897.id21121.diff
Thu, Oct 17, 4:55 AM
F13969949: D8897.id21121.diff
Thu, Oct 17, 4:51 AM
F13969632: D8897.id21113.diff
Thu, Oct 17, 3:28 AM
Subscribers

Details

Summary

Fixes T4913. We silently accept git and normal URIs with leading spaces, but other systems will not accept these. Instead, fail to parse these so they don't get into the system if we're validating URIs.

Test Plan

Screen_Shot_2014-04-29_at_10.33.52_AM.png (956×1 px, 122 KB)

Diff Detail

Repository
rPHU libphutil
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley retitled this revision from to Make PhutilGitURI and PhutilURI strict about leading whitespace.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.

Ah - thanks for discussion in T4913.

This revision is now accepted and ready to land.Apr 29 2014, 5:47 PM
epriestley updated this revision to Diff 21121.

Closed by commit rPHU1490f4fa10c8 (authored by @epriestley).