See #31641 for details
There is a bug in com_csp in that it is only showing the domain root which is not the same as the site root AND it is not showing the actual url
Labels |
Added:
?
|
Labels |
Added:
?
|
Labels |
Added:
?
|
Solution is here: #31641 (comment) ;-)
We just need to have someone with the time to make a PR with that changes.
Two things I don't like at the solution:
parse_url
instead of Uri
(internal checks etc.)Agree feel free to send a PR that uses uri vs parse_url.
Wildly saving a full url with all parameters looks for me like a good way to open up a gate for potential XSS as any URL can be send in the request + there is a high potential that an admin opens the url to check the page for the stuff, because the list is really lacking important information (which script etc)
Agree on the XSS stuff any ideas to solve it?
There is an issue open on the CSP specs site. Right now we could only get a short part of the script but not the full or even better the script hash to whitelist it.
w3c/webappsec-csp#378
With
Agree thats the reason i proposed the change that still allows the full url to be shown as before the other PR.
Status | New | ⇒ | Closed |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-03-29 08:44:07 |
Closed_By | ⇒ | rdeutz | |
Labels |
Added:
?
?
Removed: ? ? |
Just for documentation, as I'm not sure why it's done that way:
here the full URL is available but only the domain is saved. As @zero-24 did a lot of changes here, perhaps he can explain?