Skip to content

Commit

Permalink
refactor: set max_header_size to the Apache default of 8190
Browse files Browse the repository at this point in the history
  • Loading branch information
lsiebels committed Aug 8, 2024
1 parent 9bc6481 commit b8e8ef4
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 4 deletions.
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,12 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

### Fixed

## [1.0.4] - 2024-08-08

### Changed

- Changed max_header_size from 8000 to the Apache default of 8190.

## [1.0.3] - 2024-08-07

### Added
Expand All @@ -35,6 +41,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

## [1.0.0] - 2024-08-02

[1.0.4]: https://github.com/basecom/magento2-csp-split-header/compare/v1.0.3...v1.0.4
[1.0.3]: https://github.com/basecom/magento2-csp-split-header/compare/v1.0.2...v1.0.3
[1.0.2]: https://github.com/basecom/magento2-csp-split-header/compare/v1.0.1...v1.0.2
[1.0.1]: https://github.com/basecom/magento2-csp-split-header/compare/v1.0.0...v1.0.1
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ no longer exceed the maximum permitted length of the web server.
| Config | Default Value | Description |
|-------------------------------------------------------------|----------------|------------------------------------------------------------|
| `basecom_csp_split_header/settings/header_splitting_enable` | 0 _(disabled)_ | enables (1) / disables (0) the splitting of the CSP header |
| `basecom_csp_split_header/settings/max_header_size` | 8000 | maximum allowed header size |
| `basecom_csp_split_header/settings/max_header_size` | 8190 | maximum allowed header field size |

These values can be updated in the system configuration under `Basecom -> Content Security Policy -> Enable`.

Expand Down
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "basecom/magento2-csp-split-header",
"version": "1.0.3",
"version": "1.0.4",
"description": "Magento 2 module to split oversized CSP headers into multiple headers.",
"type": "magento2-module",
"license": [
Expand Down
5 changes: 3 additions & 2 deletions etc/config.xml
Original file line number Diff line number Diff line change
@@ -1,10 +1,11 @@
<?xml version="1.0"?>
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:module:Magento_Store:etc/config.xsd">
<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
xsi:noNamespaceSchemaLocation="urn:magento:module:Magento_Store:etc/config.xsd">
<default>
<basecom_csp_split_header>
<settings>
<header_splitting_enable>0</header_splitting_enable>
<max_header_size>8000</max_header_size>
<max_header_size>8190</max_header_size>
</settings>
</basecom_csp_split_header>
</default>
Expand Down

0 comments on commit b8e8ef4

Please sign in to comment.