[#39378] - Adding support to the NEL header to the core httpheaders plugin
- Fixed in Code Base
- 3 Feb 2023
- Medium
- Build: 4.2-dev
- # 39378
- Diff
- zero-24:nel
User tests: Successful: Unsuccessful:
Pull Request for Issue zero-24/plg_system_httpheader#43 (comment)
Summary of Changes
Adding support to the NEL header to the core httpheaders plugin
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/NEL
Testing Instructions
- Verify that you cant add a "NEL" header
- apply the patch
- confirm you can set the NEL header now
Actual result BEFORE applying this Pull Request
No support for the NEL header
Expected result AFTER applying this Pull Request
Now we do support the NEL header
Link to documentations
Please select:
-
Documentation link for docs.joomla.org: https://docs.joomla.org/J4.x:Http_Header_Management
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Front End Plugins |
| Status | New | ⇒ | Pending |
| Labels |
Added:
?
|
||
Shouldn't that be for 4.3-dev as it's a new feature?
A "New feature" tbh I dont think it has to go into 4.3-dev but I'm happy to move it around as soon as 4.3-dev is up-to-date with 4.2-dev branch so i can actually move it around :D
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39378.
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39378.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39378.
Please move that to 4.3-dev again. Maybe it is not a new feature, but it is not fixing wrong behaviour too :)
| Labels |
Added:
?
PR-4.3-dev
Removed: ? |
||
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-02-03 22:14:52 |
| Closed_By | ⇒ | obuisard | |
| Labels |
Added:
Documentation Required
|
||
Shouldn't that be for 4.3-dev as it's a new feature?