-
-
Notifications
You must be signed in to change notification settings - Fork 9.6k
[DependencyInjection] Added @@ escaping strategy for YamlFileLoader and YamlDumper #7357
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
Conversation
… and YamlDumper Added the possibility to to use '@@' as an escaping strategy for parameters that should be treated as strings but start with '@' (i.e. safe mailer passwords).
I'm really need this feature in 2.2 branch. |
This PR was submitted for the master branch but it was merged into the 2.1 branch instead (closes #7357). Discussion ---------- [DependencyInjection] Added @@ escaping strategy for YamlFileLoader and YamlDumper Added the possibility to to use ```@@``` as an escaping strategy for parameters that should be treated as strings but start with ```@``` (i.e. safe mailer passwords). This PR is deprecating #7307, since as a new feature it has to be compared against the master branch. | Q | A | | ------ | ------ | | Bug fix? | yes | | New feature? | yes | | BC breaks? | no | | Deprecations? | no | | Tests pass? | yes | | Fixed tickets | #4889 | Commits ------- 8cdf387 [DependencyInjection] Added '@@' escaping strategy for YamlFileLoader and YamlDumper
This has been merged in 2.1 as this is really more a bug fix than a new feature. |
This PR deprecates symfony#2286 | Q | A | ------------- | --- | Doc fix? | no | New docs? | yes symfony/symfony#7357 | Applies to | 2.1 | Fixed tickets | symfony/symfony#4889
This doesn't work if the password is @MyPasswordIs@Right123@# Error: "InvalidTypeException: Invalid type for path "doctrine.dbal.connections.default.password". Expected scalar, but got object." |
@privacybob Use quotes. |
@stloyd I did, here's my parameters.yml: parameters: |
I just realized that the password needs to have two @ signs like this: @@MyPasswordIs@@right123@@# Above password works. |
@privacybob Yes, you can find the docs about that here: http://symfony.com/doc/current/book/service_container.html#service-parameters And only the |
Thanks @tPl0ch |
Added the possibility to to use
@@
as an escaping strategy forparameters that should be treated as strings but start with
@
(i.e. safe mailer passwords).
This PR is deprecating #7307, since as a new feature it has to be compared against the master branch.