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

$ signs not escaped when retrieving password from deploy conf file #3421

Closed
DavidW250 opened this issue Feb 22, 2021 · 2 comments
Closed

$ signs not escaped when retrieving password from deploy conf file #3421

DavidW250 opened this issue Feb 22, 2021 · 2 comments

Comments

@DavidW250
Copy link

I deploy certificates on a Synology NAS using the synology_dsm deploy hook. The hook calls _getdeployconf() to retrieve the admin password stored in the deploy configuration file:

_getdeployconf SYNO_Password

_getdeployconf is not properly escaping '$' symbols when it retrieves SAVED_SYNO_Password from the config file and the '$' plus the following character are being stripped from the password.

@Run-King
Copy link

Run-King commented May 2, 2021

I just ran into this issue as well.
It seems this can be fixed by replacing line https://github.com/acmesh-official/acme.sh/blob/master/acme.sh#L2269
eval "export $_rac_key=\"$_saved\""
with
eval "export $_rac_key=\"\$_saved\""
This way the _saved variable will be evaluated instead of it's value.

@DavidW250
Copy link
Author

Thank you Run-King. Although I could figure out where the problem was, I wasn't knowledgeable enough to figure out how to fix it without potentially breaking something.

Sp1l pushed a commit to Sp1l/acme.sh that referenced this issue Aug 10, 2021
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

2 participants