Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.2k
Improved the explanation about the "secret" configuration parameter#5109
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Improved the explanation about the "secret" configuration parameter#5109
Uh oh!
There was an error while loading.Please reload this page.
Conversation
javiereguiluz commentedMar 24, 2015
| Q | A |
|---|---|
| Doc fix? | no |
| New docs? | yes |
| Applies to | all |
| Fixed tickets | - |
dupuchba commentedMar 24, 2015
👍 |
| and for creating signed URIs when using :ref:`ESI (Edge Side Includes) <edge-side-includes>` . | ||
| This option becomes the service container parameter named ``kernel.secret``, | ||
| which you can use whenever the application needs a immutable random string |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
an immutable
| which you can use whenever the application needs an immutable random string | ||
| to add more entropy. | ||
| As with any other security-related parameter, is a good practice to change this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
"[...]parameter, it is a [...]"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Fixed. Thanks.
👍 to this. The sentence |
@ricardclau you are right about that sentence. It sounds awful. I've just reworded it. Thanks Ricard. |
Wonderful explanation! Thanks Javier! |
…on parameter (javiereguiluz)This PR was merged into the 2.3 branch.Discussion----------Improved the explanation about the "secret" configuration parameter| Q | A| ------------- | ---| Doc fix? | no| New docs? | yes| Applies to | all| Fixed tickets | -Commits-------ac3a74d Minor rewordingdb5db0f Fixed a minor grammar issue74841e6 Minor rewording4bbd84a Fixed some typosb13250d Improved the explanation about the "secret" configuration parameter