fix(deps): update module github.com/go-acme/lego/v4 to v5#724
fix(deps): update module github.com/go-acme/lego/v4 to v5#724gardener-ci-robot wants to merge 8 commits into
Conversation
55d462d to
bdb76bb
Compare
bdb76bb to
5a23fdf
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
f552350 to
272166b
Compare
|
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
95d7494 to
7f78a09
Compare
|
/hold needs more testing |
|
/unhold also tested with precheck namespaces |
|
/cc @marc1404 |
2b00c93 to
adb9583
Compare
|
rebased to resolve merge conflict |
|
/cla |
|
Successfully reached out to cla-assistant.io to initialize recheck of PR #724 |
marc1404
left a comment
There was a problem hiding this comment.
Thank you for taking care of doing all the necessary steps to adapt to the changes in v5 of the library! 🙏
With the help of Claude or not, I appreciate the added test that covers the v4->v5 migration path: 17c7515 (this PR)
A few comments below 👇
| // parseNameservers ensures that all nameservers have a port number, and adds the default DNS port 53 if not. | ||
| // This function replicates functionality that was removed from lego v5. | ||
| func parseNameservers(servers []string) []string { | ||
| var resolvers []string | ||
| for _, resolver := range servers { | ||
| // ensure all servers have a port number | ||
| if _, _, err := net.SplitHostPort(resolver); err != nil { | ||
| resolvers = append(resolvers, net.JoinHostPort(resolver, "53")) | ||
| } else { | ||
| resolvers = append(resolvers, resolver) | ||
| } | ||
| } | ||
| return resolvers | ||
| } |
There was a problem hiding this comment.
This function could be covered by unit tests
There was a problem hiding this comment.
This is indirectly already covered by unit tests for the PreparePrecheckNameservers method.
A few more test cases have been added for IPv6.
…r ACME registration (with help of claude)
adb9583 to
83bd76d
Compare
83bd76d to
250f380
Compare
|
rebased to resolve merge conflict |
This PR contains the following updates:
v4.35.2→v5.0.4Release Notes
go-acme/lego (github.com/go-acme/lego/v4)
v5.0.4Compare Source
Fixed
v5.0.3Compare Source
Fixed
Added
v5.0.2Compare Source
Fixed
v5.0.1Compare Source
Fixed
v5.0.0Compare Source
Added
Changed
crypto.PrivateKeywithcrypto.SignerRemoved
googledomains,azure(replaced byazuredns),cloudxns,dnspod,brandit,iwantmyname,iij(replaced byiijdpf).Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate.