Page MenuHomePhabricator
Differential D19218

Fix the DarkConsole inline error log stack trace expansion behavior for Content-Security-Policy
ClosedPublic
Actions

Authored by epriestley on Mar 13 2018, 11:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 8:49 AM
Unknown Object (File)
Thu, Apr 11, 8:37 AM
Unknown Object (File)
Sat, Apr 6, 4:46 AM
Unknown Object (File)
Sun, Mar 31, 7:32 AM
Unknown Object (File)
Sun, Mar 31, 2:32 AM
Unknown Object (File)
Sun, Mar 31, 2:32 AM
Unknown Object (File)
Sun, Mar 31, 2:32 AM
Unknown Object (File)
Mar 10 2024, 7:08 PM
View All 90 Files
Subscribers
None

Details

Reviewers
None
Maniphest Tasks
T13102: Plans: 2018 Week 11 Bonus Content
Commits
rPdc7e40ff3fc7: Fix the DarkConsole inline error log stack trace expansion behavior for Content…
Summary

See PHI451. Ref T13102. DarkConsole uses an ancient inline "onclick" handler to expand the stack traces for errors.

The new Content-Security-Policy prevents this from functioning.

Replace this with a more modern behavior-driven action instead.

Test Plan
  • Clicked some errors in DarkConsole, saw stack traces appear.
  • Grepped for onclick and jsprintf() to see if I could find any more of these, but came up empty.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 13 2018, 11:39 PM
Harbormaster completed remote builds in B19837: Diff 46020.Mar 13 2018, 11:40 PM
epriestley requested review of this revision.Mar 13 2018, 11:40 PM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 13 2018, 11:45 PM
Closed by commit rPdc7e40ff3fc7: Fix the DarkConsole inline error log stack trace expansion behavior for Content… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
celerity/
34 lines
4 lines
src/
applications/
console/
plugin/
7 lines
view/
page/
3 lines
webroot/
rsrc/
js/
core/
darkconsole/
20 lines

Diff 46021

View Options

resources/celerity/map.php

Loading...
View Options

resources/celerity/packages.php

Loading...
View Options

src/applications/console/plugin/DarkConsoleErrorLogPlugin.php

Loading...
View Options

src/view/page/PhabricatorStandardPageView.php

Loading...
View Options

webroot/rsrc/js/core/behavior-error-log.js

Loading...
View Options

webroot/rsrc/js/core/darkconsole/behavior-dark-console.js

Loading...
Phabricator · Built by Phacility