Allow setting the proxy through an environment variable
See original GitHub issueIs your proposal related to a problem?
We have multiple tenants and the tenant is specified as part of the url that we proxy to
e.g. The bindings we use locally are of the form tenant1.localhost, tenant2.localhost
Currently we manually change the value of the proxy
setting in the package.json file depending on which tenant we’re testing against. This then means that that change often gets committed and leads to unnecessary conflicts as other devs have done the same.
Describe the solution you’d like
I suggest allowing the proxy setting to be controlled by either the package.json or an environment variable setting, with the environment variable winning if both are set. This would mean that a default value could be stored in the package.json, for the first time a dev environment is set up, but can be easily overridden without the possibilty for committing changes that lead to conflicts or affect other dev environments.
Describe alternatives you’ve considered
I tried configuring the proxy manually but we don’t have a simple set of rules that can be applied to determine if the request should be proxied, so I was unsuccessful, and the existing react scripts one works perfectly for our needs apart from this one issue.
Additional context
I think that the proxy
setting is similar to other advanced configuration options set using environment variables in that they are often specific to development environments and so should be configurable in the same way.
Issue Analytics
- State:
- Created 3 years ago
- Reactions:10
- Comments:9
Top GitHub Comments
Agreed! The proxy, like the host, port etc isn’t a concern of the package.json. It should live in environment variables.
I just came looking to find if this feature exists already. Having to change the proxy in
package.json
, a VCS tracked file is not ideal for temporary adjustments that I don’t want to commit. Making changes in.env.development.local
would be much preferred!Hopefully the PR can be merged soon!