Skip to content
This repository has been archived by the owner on Jul 4, 2018. It is now read-only.

Allowed shared ContextListener for firewalls #964

Closed
wants to merge 2 commits into from
Closed

Allowed shared ContextListener for firewalls #964

wants to merge 2 commits into from

Conversation

ibrasho
Copy link
Contributor

@ibrasho ibrasho commented Jun 20, 2014

In response to #963.

Checks for a context key in the firewall configuration, if it's not there use name.

@@ -171,7 +171,9 @@ public function register(Container $app)
$users = isset($firewall['users']) ? $firewall['users'] : array();
$security = isset($firewall['security']) ? (bool) $firewall['security'] : true;
$stateless = isset($firewall['stateless']) ? (bool) $firewall['stateless'] : false;
unset($firewall['pattern'], $firewall['users'], $firewall['security'], $firewall['stateless']);
$stateless = isset($firewall['stateless']) ? (bool) $firewall['stateless'] : false;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicates previous line.

@ibrasho
Copy link
Contributor Author

ibrasho commented Jun 20, 2014

@rybakit Fixed, thanks. 😄

@fabpot
Copy link
Member

fabpot commented Jun 30, 2014

Thank you @ibrasho.

@fabpot fabpot closed this in 215251c Jun 30, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants