Differential D18149

Fix ambiguous URI parsing in Youtube Remarkup rule
ClosedPublic
Actions

Authored by epriestley on Jun 23 2017, 2:37 PM.

Tags

None

Referenced Files

	F18181389: D18149.id43671.diff
	Sat, Aug 16, 8:35 AM

	F18178039: D18149.id43668.diff
	Sat, Aug 16, 12:56 AM

	F18124705: D18149.id43671.diff
	Wed, Aug 13, 12:43 PM

	F18124703: D18149.id43668.diff
	Wed, Aug 13, 12:43 PM

	F18124700: D18149.id.diff
	Wed, Aug 13, 12:43 PM

	F18112162: D18149.diff
	Tue, Aug 12, 4:06 PM

	F18099953: D18149.id43671.diff
	Sat, Aug 9, 9:44 AM

	F18094762: D18149.id.diff
	Thu, Aug 7, 10:20 PM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T12867: Youtube remarkup rule fails to parse "ambiguous URI"

Commits

rP988a52cf1acd: Fix ambiguous URI parsing in Youtube Remarkup rule

Summary

Fixes T12867. Also:

Simplify the code a little.
Stop mutating this on text/mobile -- there's no inherent value in the "youtu.be" link so I think this just changes the text the user wrote unnecessarily.

Test Plan

Screen Shot 2017-06-23 at 7.32.50 AM.png (552×1 px, 395 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 23 2017, 2:37 PM

Harbormaster completed remote builds in B17556: Diff 43668.Jun 23 2017, 2:38 PM

chad accepted this revision.Jun 23 2017, 3:10 PM

This revision is now accepted and ready to land.Jun 23 2017, 3:10 PM

Closed by commit rP988a52cf1acd: Fix ambiguous URI parsing in Youtube Remarkup rule (authored by epriestley, committed by epriestley). · Explain WhyJun 23 2017, 3:43 PM

This revision was automatically updated to reflect the committed changes.

epriestley mentioned this in T12526: parse_url() behavior has changed with PHP7, causing libphutil unit tests to fail and possibly creating security concerns.Sep 30 2017, 10:32 PM

epriestley mentioned this in D18666: Fix an exception in the hyperlink remarkup rule for unparseable URIs.Oct 2 2017, 2:14 PM

epriestley mentioned this in rPHU20db87e17679: Fix an exception in the hyperlink remarkup rule for unparseable URIs.Oct 2 2017, 4:10 PM

epriestley mentioned this in rPHUf7b0855acf5a: (stable) Fix an exception in the hyperlink remarkup rule for unparseable URIs.

Revision Contents
Changeset List

Path

Size

src/

infrastructure/

markup/

rule/

PhabricatorYoutubeRemarkupRule.php

30 lines

Diff 43671

src/infrastructure/markup/rule/PhabricatorYoutubeRemarkupRule.php

Loading...