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

vultr: bump client from 2.7.1 to 2.16.0 #1640

Merged
merged 1 commit into from
May 7, 2022
Merged

Conversation

ddymko
Copy link
Contributor

@ddymko ddymko commented May 7, 2022

I noticed that GoVultr hasn't been updated in a while so I went ahead and bumped to the latest version 2.16.0. No major changes have been made to DNS so behavior has stayed the same.

Some test output

$  lego [vultr-updates] make build
BIN_OUTPUT: dist/lego
rm -rf dist/ builds/ cover.out
Version: a77d023488dac3abc6bc9cbdf5850d51d88e2f55
go build -trimpath -ldflags '-X "main.version=a77d023488dac3abc6bc9cbdf5850d51d88e2f55"' -o  dist/lego ./cmd/lego/


$  dist [vultr-updates] ./lego -m [email protected] --dns vultr -d '*.dymko.dev' -d 'dymko.dev' --dns-timeout 100 -s https://acme-staging-v02.api.letsencrypt.org/directory  run
2022/05/07 12:07:02 [INFO] [*.dymko.dev, dymko.dev] acme: Obtaining bundled SAN certificate
2022/05/07 12:07:03 [INFO] [*.dymko.dev] AuthURL: https://acme-staging-v02.api.letsencrypt.org/acme/authz-v3/2387104594
2022/05/07 12:07:03 [INFO] [dymko.dev] AuthURL: https://acme-staging-v02.api.letsencrypt.org/acme/authz-v3/2387104604
2022/05/07 12:07:03 [INFO] [*.dymko.dev] acme: use dns-01 solver
2022/05/07 12:07:03 [INFO] [dymko.dev] acme: Could not find solver for: tls-alpn-01
2022/05/07 12:07:03 [INFO] [dymko.dev] acme: Could not find solver for: http-01
2022/05/07 12:07:03 [INFO] [dymko.dev] acme: use dns-01 solver
2022/05/07 12:07:03 [INFO] [*.dymko.dev] acme: Preparing to solve DNS-01
2022/05/07 12:07:03 [INFO] [dymko.dev] acme: Preparing to solve DNS-01
2022/05/07 12:07:03 [INFO] [*.dymko.dev] acme: Trying to solve DNS-01
2022/05/07 12:07:03 [INFO] [*.dymko.dev] acme: Checking DNS record propagation using [8.8.8.8:53]
2022/05/07 12:07:05 [INFO] Wait for propagation [timeout: 1m0s, interval: 2s]
2022/05/07 12:07:05 [INFO] [*.dymko.dev] acme: Waiting for DNS record propagation.
2022/05/07 12:07:20 [INFO] [*.dymko.dev] The server validated our request
2022/05/07 12:07:20 [INFO] [dymko.dev] acme: Trying to solve DNS-01
2022/05/07 12:07:20 [INFO] [dymko.dev] acme: Checking DNS record propagation using [8.8.8.8:53]
2022/05/07 12:07:22 [INFO] Wait for propagation [timeout: 1m0s, interval: 2s]
2022/05/07 12:07:34 [INFO] [dymko.dev] The server validated our request
2022/05/07 12:07:34 [INFO] [*.dymko.dev] acme: Cleaning DNS-01 challenge
2022/05/07 12:07:34 [INFO] [dymko.dev] acme: Cleaning DNS-01 challenge
2022/05/07 12:07:34 [INFO] [*.dymko.dev, dymko.dev] acme: Validations succeeded; requesting certificates
2022/05/07 12:07:34 [INFO] [*.dymko.dev] Server responded with a certificate.

@ldez ldez added this to the v4.7 milestone May 7, 2022
@ldez ldez changed the title Vultr: govultr from 2.7.1 to 2.16.0 vultr: bump client from 2.7.1 to 2.16.0 May 7, 2022
Copy link
Member

@ldez ldez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ldez ldez enabled auto-merge (squash) May 7, 2022 16:11
@ldez ldez merged commit fdb11d7 into go-acme:master May 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants