Skip to content
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

Fields with spaces in dotenv #38

Open
ghost opened this issue Nov 25, 2021 · 1 comment
Open

Fields with spaces in dotenv #38

ghost opened this issue Nov 25, 2021 · 1 comment

Comments

@ghost
Copy link

ghost commented Nov 25, 2021

Dotenv files commonly contain string literals with spaces due to bash field splitting. The typical format of an env file to prevent bash field splitting is to wrap the string with spaces like:

EAUTH_BANNER="Some Long String"
EAUTH_MESSAGE_PREFIX="Authenticate with your MetaMask wallet" 

In contract.js the above format causes issues when reading with process.env as the result is "Some long string" and not Some long string. I propose to add some handling logic in contract.js to handle this common case for better interoperability with bash scripts.

@rht
Copy link
Contributor

rht commented Nov 25, 2021

EAUTH_MESSAGE_PREFIX="Authenticate with your MetaMask wallet" is for Docker Compose v2.
EAUTH_MESSAGE_PREFIX=Authenticate with your MetaMask wallet is for Docker Compose v1.

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

No branches or pull requests

1 participant