Skip to content

Commit

Permalink
Merge pull request #86 from niels-nijens/security-listener-priority
Browse files Browse the repository at this point in the history
Change priority of RequestValidationSubscriber to give more priority to the firewall listeners of Symfony's security bundle
  • Loading branch information
niels-nijens committed Mar 20, 2024
2 parents 6c8af03 + de3379e commit 2faf0f2
Show file tree
Hide file tree
Showing 9 changed files with 580 additions and 4 deletions.
5 changes: 5 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,13 +17,18 @@ ifndef version
@exit 1
endif

ifeq ($(filter $(version),5.3 5.4 6.0 6.1),)
sed -i -e "s/\(\s\+\)\(enable_authenticator_manager:\)/\1# \2/" tests/Functional/App/config.yaml
endif

ifeq ($(filter $(version),6.2 6.3 6.4),)
sed -i -e "s/\(\s\+\)\(handle_all_throwables:\)/\1# \2/" tests/Functional/App/config.yaml
endif

ifeq ($(filter $(version),5.3 5.4 6.0 6.1 6.2 6.3 6.4),)
sed -i -e "s/\(\s\+\)# \(storage_id:\)/\1\2/" tests/Functional/App/config.yaml
sed -i -e "s/\(\s\+\)\(storage_factory_id:\)/\1# \2/" tests/Functional/App/config.yaml
sed -i -e "s/\(\s\+\)\(lazy:\)/\1# \2/" tests/Functional/App/config.yaml
endif

composer global config --no-plugins allow-plugins.symfony/flex true
Expand Down
3 changes: 2 additions & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,8 @@
"phpunit/phpunit": "^9.3",
"symfony/browser-kit": "^4.4 || ^5.0 || ^6.0",
"symfony/phpunit-bridge": "^4.4 || ^5.0 || ^6.0",
"symfony/process": "^4.4 || ^5.0 || ^6.0"
"symfony/process": "^4.4 || ^5.0 || ^6.0",
"symfony/security-bundle": "^4.4 || ^5.0 || ^6.0"
},
"autoload": {
"psr-4": {
Expand Down
Loading

0 comments on commit 2faf0f2

Please sign in to comment.