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

_checkSudo: [ -z "__INTERACTIVE" ] should be [ -z "$__INTERACTIVE" ] #5216

Closed
oittaa opened this issue Jul 20, 2024 · 1 comment
Closed

_checkSudo: [ -z "__INTERACTIVE" ] should be [ -z "$__INTERACTIVE" ] #5216

oittaa opened this issue Jul 20, 2024 · 1 comment

Comments

@oittaa
Copy link
Contributor

oittaa commented Jul 20, 2024

Steps to reproduce

In the current master version , _checkSudo() interactivity check doesn't work.

acme.sh/acme.sh

Line 7142 in d81fc15

if [ -z "__INTERACTIVE" ]; then

That line always evaluates as false.

The fix is to add the missing dollar sign and actually check the variable has been set like the following:

  if [ -z "$__INTERACTIVE" ]; then

Debug log

I believe the bug was introduced in this commit: 14b5914

Copy link

Please upgrade to the latest code and try again first. Maybe it's already fixed. acme.sh --upgrade If it's still not working, please provide the log with --debug 2, otherwise, nobody can help you.

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