Differential D19158

" in the Content Security Policy
ClosedPublic
Actions

Authored by epriestley on Mar 1 2018, 2:54 AM.

Tags

None

Referenced Files

	F13217339: D19158.diff
	Sat, May 18, 4:40 AM

	F13204927: D19158.diff
	Wed, May 15, 1:15 AM

	F13201988: D19158.diff
	Tue, May 14, 7:28 PM

	F13201429: D19158.diff
	Tue, May 14, 12:08 PM

	F13188166: D19158.diff
	Sat, May 11, 5:04 AM

	F13174245: D19158.id45894.diff
	Tue, May 7, 10:26 PM

	Unknown Object (File)
	Tue, May 7, 4:08 AM

	Unknown Object (File)
	Mon, May 6, 9:24 PM

View All 87 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T4340: Implement Content-Security-Policy and Strict-Transport-Security headers
Commits: rPd5befb1a0ea3: Block use of "<base />" in the Content Security Policy

Summary

Ref T4340. We don't use "<base />" so we can safely block it.

Test Plan

Injected "<base />" into a page, saw an error in the console showing that the browser had blocked it.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 1 2018, 2:54 AM

Harbormaster completed remote builds in B19704: Diff 45893.Mar 1 2018, 2:55 AM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 1 2018, 2:55 AM

epriestley requested review of this revision.

Closed by commit rPd5befb1a0ea3: Block use of "<base />" in the Content Security Policy (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

aphront/

response/

AphrontResponse.php

3 lines

Diff 45894

View Options

src/aphront/response/AphrontResponse.php

Block use of "<base />" in the Content Security PolicyClosedPublicActions