Comments (4)
The LoggerInterface is deprecated since Symfony 2.2 but it's possible there are still packages out there that are using this interface (and this bundle still supports Symfony 2.3).
So instead of updating the typehint, it should be removed and we need check if eg. an instance of Psr\Log\LoggerInterface
or Symfony\Component\HttpKernel\Log\LoggerInterface
is provided.
There is only one one line where the logger is used: https://github.com/nelmio/NelmioSecurityBundle/blob/master/EventListener/ExternalRedirectListener.php#L81 and this needs to check which interface is used (at least until support for Symfony 2.x is dropped).
from nelmiosecuritybundle.
@VuiLenDi for now (totally temporarily) you may use my repository:
https://github.com/gencer/nelmio-symfony3
$ compose require "gencer/nelmio-symfony3"
from nelmiosecuritybundle.
Should be fixed now, I just migrated to PSR-3 logger because frankly 2.2 is very old, and this would only affect someone implementing their own logger against this outdated interface, while most people use monolog as it's been shipping with symfony2 since day one. I think it's safe enough and if not it should be easy for people to upgrade anyway.
from nelmiosecuritybundle.
Thanks @Seldaek
from nelmiosecuritybundle.
Related Issues (20)
- Ability to override (and disable) `hash` HOT 3
- Symfony 6 compatibility HOT 3
- [3.0] Roadmap HOT 1
- [RFC] Remove Cookie Session Handler in 3.0 HOT 1
- Can't use environment variables to enable/disable features HOT 2
- Problem with latest Symfony update and signed_cookie feature HOT 5
- Incompatible with Symfony's clearCookie() strategy HOT 1
- cspscript for encore_entry_script_tags and cspstyle for encore_entry_link_tags HOT 2
- Issue with CSP config HOT 2
- Remove "Server" header HOT 1
- Reusing nonce for Turbo Drive integration HOT 1
- Update the description of this GitHub repository HOT 2
- Update the default hashing algorithm used in signed cookies
- DirectiveSet::canNotBeFallbackedByDefault(): Argument #2 ($value) must be of type string, bool given
- Introduce `ExternalRedirectResponse` for more fine-grained redirection control
- When script-src is set to strict-dynamic, 2 nonces are listed in CSP header HOT 1
- CSP interferes with var-dumper HOT 1
- Nonce is empty unless dump using twig HOT 4
- [3.2.0] The report-uri directive has been deprecated HOT 2
- Incompatibility with twig 3.9 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nelmiosecuritybundle.