Paths

Table of Contentst

Filter "javascript:" URIs more aggressively
ClosedPublic
Actions

Authored by epriestley on Mar 16 2014, 2:25 PM.

Tags

None

Referenced Files

	F19506391: D8548.id20293.diff
	Fri, Jan 9, 4:45 PM

	F19095531: D8548.diff
	Dec 4 2025, 5:29 AM

	F18866785: D8548.id.diff
	Nov 3 2025, 7:22 PM

	F18817655: D8548.id20282.diff
	Oct 21 2025, 5:18 PM

	F18805056: D8548.id20283.diff
	Oct 18 2025, 1:31 PM

	F18767665: D8548.id20293.diff
	Oct 7 2025, 11:40 PM

	F18756997: D8548.id20281.diff
	Oct 5 2025, 2:57 PM

	F18748268: D8548.id.diff
	Oct 3 2025, 8:18 PM

View All Files

Subscribers

epriestley

Details

Reviewers

btrahan

Commits

rPHU9678247c9f4c: Filter "javascript:" URIs more aggressively

Summary

Some browsers interpret javascript\n: as a Javascript URI. Be significantly more aggressive in normalizing URIs when looking for javascript: hrefs.

Test Plan

Added and executed unit tests.

Diff Detail

Repository

rPHU libphutil

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

epriestley updated this revision to Diff 20281.Mar 16 2014, 2:25 PM

epriestley retitled this revision from to Filter "javascript:" URIs more aggressively.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

Herald added a subscriber: epriestley.Mar 16 2014, 2:25 PM

Add a test case for not normalizing capital letters away.

Add a test case for another true negative.

btrahan accepted this revision.Mar 17 2014, 6:12 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Mar 17 2014, 6:12 PM

Closed by commit rPHU9678247c9f4c (authored by @epriestley).

Revision Contents
Changeset List

Path

Size

src/

markup/

__tests__/

PhutilMarkupTestCase.php

33 lines

render.php

33 lines

Diff 20293

View Options

src/markup/tests/PhutilMarkupTestCase.php

View Options

Filter "javascript:" URIs more aggressivelyClosedPublicActions