Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove $ char from pw generation #12291

Open
etj opened this issue May 30, 2024 · 0 comments · May be fixed by #12427
Open

Remove $ char from pw generation #12291

etj opened this issue May 30, 2024 · 0 comments · May be fixed by #12427

Comments

@etj
Copy link
Contributor

etj commented May 30, 2024

$ char in the .env file confuses docker that will use it as part of a variable.
In order to escape $ in the .env file it should be doubled ($$), but this is reported not working in many situations.

Quick solution would be to avoid generating dollar signs when creating passwords, by removing it in this line:

geonode/create-envfile.py

Line 48 in 7ef2e13

string.ascii_letters + string.digits + string.punctuation.replace('"', "").replace("'", "").replace("`", "")

@mattiagiupponi mattiagiupponi linked a pull request Jul 31, 2024 that will close this issue
[Fixes #12291] Update create-envfile.py #12427
Open
12 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Fixes #12291] Update create-envfile.py amithkashyaps-ui/geonode
1 participant
@etj