Before reading the code, I was expecting this to be a processor to do aparse_str().
Now that I read it I see it's forparse_url().
I think we should ship both in this PR because they are complementary - that will force us to find the least confusing names for each :)

@jderusse even I appreciate your contribution, I'm going to share an (unpopular?) opinion about this "env processor" trend. I think it's a mistake and makes the YAML config files even harder to understand. I know we need to support something like this ... but I prefer if we focused instead on switching to PHP config by default.

Before

mongo_db_bundle:clients:default:hosts:                -{ host: '%env(key:host:url:MONGODB_DSN)%', port: '%env(key:port:url:MONGODB_DSN)%' }username:'%env(key:user:url:MONGODB_DSN)%'password:'%env(key:pass:url:MONGODB_DSN)%'connections:default:database_name:'%env(key:path:url:MONGODB_DSN)%'

After

$mongo =parse_url(env('MONGODB_DSN'));return ['mongo_db_bundle' => ['clients' => ['default' => ['hosts' => ['host' =>$mongo['host'],'port' =>$mongo['port'],            ],'username' =>$mongo['user'] ??null,'password' =>$mongo['pass'] ??null,        ]    ],'connections' => ['default' => ['database_name' =>$mongo['path'],        ]    ]];

parse_url(env('MONGODB_DSN'));

fundamentally the goal of %env()% is to not fetch envs during compile time.

But for PHP config a env builder might be nice:

'database_name' =>env('key:path','url','MONGODB_DSN'),

to at least make the strings more readable.

@javiereguiluz That is a very popular opinion to me :)

@ro0NL Isn't it possible to have something like the code below?

// services.phpuseSymfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;returnfunction (ContainerConfigurator$configurator) {$parameters =$configurator->parameters();$defaultMongoDbUrl ='mongodb://localhost:27017/symfony?ssl=false';$parameters->set('database_name',env('MONGODB_DSN',$defaultMongoDbUrl)->url()->path()    );$parameters->set('database_secure',env('MONGODB_DSN',$defaultMongoDbUrl)->url()->query('ssl',true/* default */)    );// other examples$parameters->set('security_secret',/* without default value, it will throw exception if does not exist at runtime */env('APP_SECRET_FILE')->file()->trim()     );$parameters->set('locale',env('APP_LOCALE','pt_BR'));/* defaut values are always a string */$parameters->set('account_threshold',env('APP_THRESHOLD','10')->int());};

also for the other PR#28976

@andersonamuller sure, whatever API that can build the string for us :)

the question is; how much code do we want to add/maintain only to generate a string.

@ro0NL I once opened a RFC for this#28084

@javiereguiluz of course we can do everything with php config file. Does Symfony want to promote use of yaml config file for simple case and php for complex one. This is not my call ;-) But in that case, what is the purpose of envProcessor like 'key', 'json', 'file'?

@nicolas-grekas done. Tookurl andquery.

Just to clarify: I think that YAML is fantastic for config files. The problem is when you try to add PHP-like features to YAML. If we continue doing that, it's better to switch to PHP.

@javiereguiluz please don't hijack the topic,@ro0NL is right here: your proposal misses a lot of context to have it work - maybe we can make it work, but that's a huge task, that's what I mean.

@jderusse your before/after example is perfect for Symfony Docs. Just asking about the missing parts:

• Would this feature require to enable/configure anything when using it in a Symfony full-stack app? (I guess it won't)
• Would this require some changes when using it as a stand-alone component? I guess it does ... please show some example of how to activate this. Thanks!

@javiereguiluz This feature uses exactly the same "workflow" thankey,json,int,bool, orfile processor. It don't use new Class or thing to instantiate.

You can use it without the framework like this:

useSymfony\Component\DependencyInjection\ContainerBuilder;$containerBuilder =newContainerBuilder();$containerBuilder->setParameter('es_url','%env(url:ES_HOST)%');$containerBuilder->compile(true);var_dump($containerBuilder->getParameter('es_url'));

ES_HOST='http://www.google.com' php index.phparray(8) {  ["scheme"]=>  string(4)"http"  ["host"]=>  string(14)"www.google.com"  ["port"]=>  int(0)  ["user"]=>  string(0)""  ["pass"]=>  string(0)""  ["path"]=>  string(0)""  ["query"]=>  string(0)""  ["fragment"]=>  string(0)""}

@jderusse thanks for the added examples. That's all we need for the docs. Cheers!

(rebase needed)
@symfony/deciders OK for you?

Please rebase.

Reading the updated description of this PR, I think this:

charset: '%env(key:charset:query_string:key:query:url:DATABASE_DSN)%'

could be easier to understand as follows:

charset: '%env(key:charset:query_string:DATABASE_DSN)%'

query_string would extract the query string from the given URL and explode it into a PHP array.

can we try both? query string obtained from URL and provided as-is.

@javiereguiluz I'm not sure that's possible without restricting possibilities a lot. Power comes from combining single predictable tools, and have each tool do one thing well. Having query_string be a hybrid betweenparse_url andparse_str means not being able to use it only as aparse_str unit. 👎 to your proposal for this reason on my side.

Here's an approach that might work:https://3v4l.org/hLlIg, i like the proposed convenience ofkey:charset:query_string:VAR

Thanks@ro0NL
@jderusse OK to implement this? Works for me (canceling my previous comment)

Done

Thank you@jderusse.