Page MenuHomePhabricator

Fix an exception in the hyperlink remarkup rule for unparseable URIs
ClosedPublic

Authored by epriestley on Oct 2 2017, 2:14 PM.
Tags
None
Referenced Files
F15423772: D18666.id44817.diff
Sat, Mar 22, 6:08 PM
F15420414: D18666.id.diff
Fri, Mar 21, 1:55 PM
F15417966: D18666.diff
Thu, Mar 20, 7:46 PM
F15404192: D18666.diff
Tue, Mar 18, 6:42 AM
F15400539: D18666.id.diff
Mon, Mar 17, 12:24 PM
F15376278: D18666.id44820.diff
Thu, Mar 13, 2:38 AM
F15285242: D18666.diff
Tue, Mar 4, 12:18 PM
Unknown Object (File)
Mon, Mar 3, 12:35 PM
Subscribers
None

Details

Summary

Ref T12526. URIs in the form http://x.y/#http://x.y/# can fail to parse. Catch these cases and ignore them, rather than throwing.

See also similar changes earlier, in D18149 and D18076.

Test Plan
  • Added failing test cases, made them pass.

Screen Shot 2017-10-02 at 7.14.34 AM.png (569×385 px, 33 KB)

Diff Detail

Repository
rPHU libphutil
Lint
Lint Not Applicable
Unit
Tests Not Applicable