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

Please remove wget install way #4862

Closed
jmzwcn opened this issue Nov 8, 2023 · 4 comments
Closed

Please remove wget install way #4862

jmzwcn opened this issue Nov 8, 2023 · 4 comments

Comments

@jmzwcn
Copy link

jmzwcn commented Nov 8, 2023

Please remove the following:

wget -O - https://get.acme.sh | sh -s email=[email protected]

Actually It is wrong. ACME reqiure "curl".

Copy link

github-actions bot commented Nov 8, 2023

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.

@Neilpang Neilpang closed this as completed Nov 9, 2023
@jmzwcn
Copy link
Author

jmzwcn commented Nov 10, 2023

This is serious mistake for user with wget but NO curl. For instance, all env based alpine.

@Neilpang
Copy link
Member

https://get.acme.sh/
在安装时都判断过, 有curl 就用 curl. 没有就用wget.
我在一个新虚机中试了, 只有安装有wget 时, 完全没问题.

如果你遇到问题, 具体说问题, 正常汇报.

@jmzwcn
Copy link
Author

jmzwcn commented Nov 13, 2023

You could use nginx:alpine-slim for a try.

docker run --rm -it nginx:alpine-slim sh

apk add openssl
wget -O - https://get.acme.sh/ | sh -s [email protected]

export CF_Key=*** 
export [email protected]
/root/.acme.sh/acme.sh --issue -d test.com --dns dns_cf

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