Skip to content

Commit

Permalink
minor #20301 [SecurityBundle] Changed encoder configuration example t…
Browse files Browse the repository at this point in the history
…o bcrypt (jeremyFreeAgent)

This PR was submitted for the master branch but it was merged into the 2.7 branch instead (closes #20301).

Discussion
----------

[SecurityBundle] Changed encoder configuration example to bcrypt

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

Simple change in the configuration example to help developers to not use `sha512` as encoder when using `config:dump-reference`.

Commits
-------

a55058f [SecurityBundle] Changed encoder configuration example to bcrypt
  • Loading branch information
fabpot committed Oct 30, 2016
2 parents 0572866 + a55058f commit 16b29a1
Showing 1 changed file with 4 additions and 5 deletions.
Expand Up @@ -407,11 +407,10 @@ private function addEncodersSection(ArrayNodeDefinition $rootNode)
->children()
->arrayNode('encoders')
->example(array(
'Acme\DemoBundle\Entity\User1' => 'sha512',
'Acme\DemoBundle\Entity\User2' => array(
'algorithm' => 'sha512',
'encode_as_base64' => 'true',
'iterations' => 5000,
'AppBundle\Entity\User1' => 'bcrypt',
'AppBundle\Entity\User2' => array(
'algorithm' => 'bcrypt',
'cost' => 13,
),
))
->requiresAtLeastOneElement()
Expand Down

0 comments on commit 16b29a1

Please sign in to comment.