(stable) Don't fatal when encountering [[ <bad URI > | ... ]]


(stable) Don't fatal when encountering ...

Fixes T12796. In D17647, the parser became more strict, but this remarkup rule doesn't deal with it gracefully.

Instead, detect when the parse failed and bail out.

Test Plan:

  • Put [[ http://good.com#u:p@evil.com/ | broken ]] into a Remarkup document without backticks.
  • Before patch: fatal ("rejecting ambiguous URI").
  • After patch: link doesn't work (which is correct), but page does.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12796

Differential Revision: https://secure.phabricator.com/D18076