? Failure

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
18 Apr 2020

Pull Request for Issue #28719

Summary of Changes

Backports the changes from #28719 to 3.x

Testing Instructions

make sure the inline css and inline js still works as exptected.

Expected result

the renderer does not manipulate the inline css and JS

Actual result

the renderer does manipulate the inline css and JS that make CSP hashbased whitelisting not possible.

Documentation Changes Required

none

avatar zero-24 zero-24 - open - 18 Apr 2020
avatar zero-24 zero-24 - change - 18 Apr 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Apr 2020
Category Libraries
avatar viocassel
viocassel - comment - 18 Apr 2020

I have tested this item successfully on 1e51b54


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28720.

avatar viocassel viocassel - test_item - 18 Apr 2020 - Tested successfully
avatar richard67 richard67 - change - 18 Apr 2020
Title
[3.x] Backport Make sure there rendere does not manipulate the inline CSS and JS
[3.x] Backport Make sure there renderer does not manipulate the inline CSS and JS
avatar richard67 richard67 - edited - 18 Apr 2020
avatar richard67 richard67 - change - 18 Apr 2020
Title
[3.x] Backport Make sure there renderer does not manipulate the inline CSS and JS
[3.x] Backport Make sure the renderer does not manipulate the inline CSS and JS
avatar richard67 richard67 - edited - 18 Apr 2020
avatar zero-24 zero-24 - change - 26 Jun 2020
Labels Added: ?
avatar wilsonge
wilsonge - comment - 11 Aug 2020

@zero-24 i think in this case i do wanna see what happens in an xhtml template. these were actually a thing at the start of Joomla 3. And in J4 i can kinda justify if that breaks and we can fix but not sure it's an option for j3

avatar zero-24
zero-24 - comment - 11 Aug 2020

Do you have a xhtml template? I don't touch the dedicated xhtml supported mention in the comments to be sure.

avatar zero-24
zero-24 - comment - 25 Aug 2020

Any update here @wilsonge ? Right now this PR does not touch the xhtml code and just the none xhtml places so this should be fine to be backported from the 4.x PR

avatar wilsonge
wilsonge - comment - 3 Jan 2021

I don't have one :( all the templates I'm using are HTML5. If you're confident just merge it

avatar HLeithner
HLeithner - comment - 3 Jan 2021

@zero-24 I would move this to 3.10 since people using csp with hashes would break there site or I'm wrong?

avatar zero-24
zero-24 - comment - 3 Jan 2021

Well why? You can already have a site that use hashes in 3.x. well not autgenerated right now as the renderer is broken..

avatar HLeithner
HLeithner - comment - 3 Jan 2021

can I add the hash to the renderer now? I didn't looked at it it's only something come into my mind

Edit: ok can't be added to this function so looks ok

avatar zero-24 zero-24 - change - 7 Apr 2021
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2021-04-07 17:19:40
Closed_By zero-24
avatar zero-24 zero-24 - close - 7 Apr 2021

Add a Comment

Login with GitHub to post a comment