merged branch h4ck3rm1k3/Undefined_Security_Http_Firewall_ContextListener3 (PR #5622)
This PR was submitted for the master branch but it was merged into the 2.0 branch instead (closes #5622). Commits ------- 538f8c7 fixed undefined variable Discussion ---------- Undefined security http firewall context listener3 Updated minimal patch, First patch from static code analysis, $value should be $token I guess. please check. --------------------------------------------------------------------------- by h4ck3rm1k3 at 2012-09-27T18:00:18Z I have reverted the admin, i could create a new branch if needed. you can just cherrypick e9bb5a8 that would be easier. --------------------------------------------------------------------------- by h4ck3rm1k3 at 2012-09-27T18:29:36Z I squashed it down to one commit. thanks for your patience and help.
This commit is contained in:
commit
33abe78e6c
@ -79,7 +79,7 @@ class ContextListener implements ListenerInterface
|
||||
$token = $this->refreshUser($token);
|
||||
} elseif (null !== $token) {
|
||||
if (null !== $this->logger) {
|
||||
$this->logger->warn(sprintf('Session includes a "%s" where a security token is expected', is_object($value) ? get_class($value) : gettype($value)));
|
||||
$this->logger->warn(sprintf('Session includes a "%s" where a security token is expected', is_object($token) ? get_class($token) : gettype($token)));
|
||||
}
|
||||
|
||||
$token = null;
|
||||
|
Reference in New Issue
Block a user