Skip to content

Conversation

@gseidel
Copy link
Contributor

@gseidel gseidel commented Jan 22, 2020

Sometimes the webserver needs to run with custom env vars, than the one used by the current process.

  • Allow to pass custom env vars to WebServerManager
  • Add test to pass PANTHER_APP_ENV to Webserver

@gseidel gseidel changed the title Pass custom env webserver Pass custom env to webserver Jan 22, 2020
@gseidel
Copy link
Contributor Author

gseidel commented Jan 22, 2020

@dunglas After more work then i thought, I fixed all the deprecation warnings, but the build still failed on some other tests. I think restart the build will help to pass all tests.

@dunglas dunglas force-pushed the pass-custom-env-webserver branch 2 times, most recently from b29baa7 to 4da0dee Compare February 18, 2020 18:03
@dunglas dunglas force-pushed the pass-custom-env-webserver branch from 4da0dee to fd57e40 Compare February 18, 2020 18:20
@dunglas dunglas merged commit 5162d5c into symfony:master Feb 18, 2020
@dunglas
Copy link
Member

dunglas commented Feb 18, 2020

Thanks @gseidel

dunglas added a commit that referenced this pull request Dec 28, 2020
* allow pass custom env parameter to webserver

* add pass panther app env test

* update changelog

* fix coding styles

* fix inherit env vars for symfony 3.4

* fix deprecation warning for inherit env vars in symfony 4.4

* remove changelog

* Prevent env vars leaking

Co-authored-by: Kévin Dunglas <dunglas@gmail.com>
nicolas-grekas pushed a commit that referenced this pull request Jan 18, 2021
* allow pass custom env parameter to webserver

* add pass panther app env test

* update changelog

* fix coding styles

* fix inherit env vars for symfony 3.4

* fix deprecation warning for inherit env vars in symfony 4.4

* remove changelog

* Prevent env vars leaking

Co-authored-by: Kévin Dunglas <dunglas@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants