Skip to content

Commit

Permalink
minor #22015 [Security] Added option to return true in the method isR…
Browse files Browse the repository at this point in the history
…ememberMeRequested (WhiteEagle88)

This PR was merged into the 2.7 branch.

Discussion
----------

[Security] Added option to return true in the method isRememberMeRequested

| Q | A |
| --- | --- |
| Branch? | 2.7 |
| Bug fix? | yes |
| New feature? | no |
| BC breaks? | no |
| Deprecations? | no |
| Tests pass? | yes |
| Fixed tickets | #21807 |
| License | MIT |
| Doc PR | - |

Replaces #21989

Commits
-------

522ec3e [Security] Added option to return true in the method isRememberMeRequested
  • Loading branch information
fabpot committed Mar 21, 2017
2 parents 35e66ae + 522ec3e commit 8ba279b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
Expand Up @@ -318,6 +318,6 @@ protected function isRememberMeRequested(Request $request)
$this->logger->debug('Did not send remember-me cookie.', array('parameter' => $this->options['remember_me_parameter']));
}

return $parameter === 'true' || $parameter === 'on' || $parameter === '1' || $parameter === 'yes';
return $parameter === 'true' || $parameter === 'on' || $parameter === '1' || $parameter === 'yes' || $parameter === true;
}
}
Expand Up @@ -251,6 +251,7 @@ public function getPositiveRememberMeParameterValues()
array('1'),
array('on'),
array('yes'),
array(true),
);
}

Expand Down

0 comments on commit 8ba279b

Please sign in to comment.