@tsteur opened this Pull Request on January 30th 2019 Member

See comments in code...

@sgiehl commented on March 11th 2019 Member

Wondering if it might be better to solve that using DI, to avoid blowing up the config with more values that are typically unused/unchanged?

@tsteur commented on March 11th 2019 Member

I changed it to DI. I think it might be interesting for quite a few users whenever they want to update without downtime basically. But it can be then also documented to configure it through DI or so.

This Pull Request was closed on March 12th 2019
Powered by GitHub Issue Mirror