Feeling Lucky
?
?
[#30281] - [4.0] Correct CSP report link
- Fixed in Code Base
- 8 Aug 2020
- Medium
- Build: 4.0-dev
- # 30281
- Diff
- SharkyKZ:j4/fix/csp-report-uri
User tests: Successful: Unsuccessful:
Summary of Changes
Corrects reporting URL when Joomla is installed in a path containing /administrator.
Testing Instructions
Install Joomla in path containing /administrator. E.g. localhost/administrator-site.
Enable CSP reporting.
Check the report-uri value of content-security-policy-report-only header.
Actual result BEFORE applying this Pull Request
report-uri http://localhost-site/index.php?option=com_csp&task=report.log&client=administrator
Expected result AFTER applying this Pull Request
report-uri http://localhost/administrator-site/index.php?option=com_csp&task=report.log&client=administrator
Documentation Changes Required
No.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
zero-24
- comment
- 5 Aug 2020
Why do you want to change this? The intention was to have that log method only in the frontend.
SharkyKZ
- comment
- 5 Aug 2020
Joomla\CMS\Uri\Uri::root() does point to frontend.
zero-24
- comment
- 5 Aug 2020
Ah, ok sorry got confused :-D
bonzani
- comment
- 5 Aug 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30281.
Quy
- comment
- 6 Aug 2020
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30281.
| Status | Pending | ⇒ | Ready to Commit |
Quy
- comment
- 6 Aug 2020
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/30281.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2020-08-08 23:36:44 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
wilsonge
- comment
- 8 Aug 2020
Thanks!
Why do you want to change this? The intention was to have that log method only in the frontend.