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
Unknown Object (File)
Fri, Nov 15, 10:10 PM
Unknown Object (File)
Sun, Nov 10, 1:44 PM
Unknown Object (File)
Tue, Oct 29, 5:22 PM
Unknown Object (File)
Oct 19 2024, 8:12 PM
Unknown Object (File)
Oct 19 2024, 6:41 PM
Unknown Object (File)
Oct 19 2024, 5:57 PM
Unknown Object (File)
Oct 18 2024, 6:49 AM
Unknown Object (File)
Oct 17 2024, 4:55 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).