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

Update .env #746

Closed
wants to merge 1 commit into from
Closed

Update .env #746

wants to merge 1 commit into from

Conversation

igerlster
Copy link

Quotes are used as password, so login was set to 'changeme' and not changeme.

Quotes are used as password, so login was set to 'changeme' and not changeme.
@antoineco
Copy link
Collaborator

That is because you are using a version of Compose we don't support, as mentioned in the warning inside the Host Setup section of the README.

Summary:

  • Versions of Compose below 1.26 have a known bug that make them interpret quotes literally.
  • We need to be conservative with our usage of quotes because versions above 1.26 do require quotes when values contain special characters — such as $&!*... — which might be interpreted. (So do other popular Unix tools that read .env files, such as direnv).
    Those characters are common in passwords, including the ones generated by Elasticsearch's password tool, which usage we recommend in the README.

As you can see, we either support a version of Compose that is over two years old, or we quote passwords to prevent our users from running into issues such as in #667.
If I have to decide between the two, I choose to support newer versions anytime.

@igerlster
Copy link
Author

Oh, Sorry. We did miss this. Thanks for the reply

@igerlster igerlster closed this Aug 3, 2022
@igerlster igerlster deleted the patch-1 branch August 3, 2022 10:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants