Skip to content

The child node “secret” at path “security.firewalls.main.remember_me” must be configured -> from code example #5765

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
JVerstry opened this issue Oct 7, 2015 · 4 comments
Labels
hasPR A Pull Request has already been submitted for this issue. Security
Milestone

Comments

@JVerstry
Copy link

JVerstry commented Oct 7, 2015

I was trying to implement the remember me feature using the documentation (http://symfony.com/doc/current/cookbook/security/remember_me.html), but I got the following error message:

The child node “secret” at path “security.firewalls.main.remember_me” must be configured

I opened a StackOverflow question at: http://stackoverflow.com/questions/32993824/the-child-node-secret-at-path-security-firewalls-main-remember-me-must-be-co/32998125.

Someone says key has been replaced with secret. I have updated my config and indeed the error is gone, but a new one appears. The documentation needs to be updated.

@xabbuh xabbuh added Security hasPR A Pull Request has already been submitted for this issue. labels Oct 7, 2015
@xabbuh xabbuh added this to the 2.8 milestone Oct 7, 2015
@xabbuh
Copy link
Member

xabbuh commented Oct 7, 2015

Thank you for reporting this @JVerstry. The good news is that we already have an open pull request that tackles this issue (see #5761).

@xabbuh
Copy link
Member

xabbuh commented Oct 7, 2015

However, using the old config option (key) should still work, but simply trigger a deprecation warning. Do you really use Symfony 2.8 or did you maybe install Symfony 3.0?

@JVerstry
Copy link
Author

JVerstry commented Oct 8, 2015

When I run my app in dev mode (app_dev.php), the version in the debug toolbar at the bottom of my page is 2.8.0-DEV. The require in my composer.json is "symfony/symfony": "2.8.x-dev".

@stof
Copy link
Member

stof commented Oct 8, 2015

There is a bug in the BC layer. See symfony/symfony#16176 for the fix

fabpot added a commit to symfony/symfony that referenced this issue Oct 11, 2015
…_me (stof)

This PR was merged into the 2.8 branch.

Discussion
----------

Fix the BC layer for the key->secret renaming for remember_me

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | symfony/symfony-docs#5765
| License       | MIT
| Doc PR        | n/a

There was a mistake in #15141 removing the configuration entirely.

Commits
-------

f52b3a0 Fix the BC layer for the key->secret renaming for remember_me
@fabpot fabpot closed this as completed Oct 11, 2015
fabpot added a commit to symfony/security-bundle that referenced this issue Oct 11, 2015
…_me (stof)

This PR was merged into the 2.8 branch.

Discussion
----------

Fix the BC layer for the key->secret renaming for remember_me

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | symfony/symfony-docs#5765
| License       | MIT
| Doc PR        | n/a

There was a mistake in symfony/symfony#15141 removing the configuration entirely.

Commits
-------

f52b3a0 Fix the BC layer for the key->secret renaming for remember_me
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hasPR A Pull Request has already been submitted for this issue. Security
Projects
None yet
Development

No branches or pull requests

4 participants