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

added custom ca provider #283

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mschneider82
Copy link

@mschneider82 mschneider82 commented Oct 12, 2021

# possible values: "letsencrypt", "letsencryptstaging", "cert", "custom", "none"
tls = "letsencryptstaging"
# only used if tls = "custom"
tls_custom_url = "https://acme-v02.example.com/directory"

@@ -1,4 +1,5 @@
//+build !test
//go:build !test

Choose a reason for hiding this comment

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

This syntax is for go1.17 and later.

Given that acme-dns still claims to compile with go1.13, the original syntax should be maintained.

Copy link
Author

Choose a reason for hiding this comment

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

this was done automaticlly by the IDE/go tools, i think the compatiblity should be ok since line 2 has the old syntax. I think its good to have both syntax supported since the old one may gets removed one day.

linuxgemini added a commit to linuxgemini/acme-dns that referenced this pull request Feb 9, 2022
Co-authored-by: Matthias Schneider <[email protected]>
Signed-off-by: İlteriş Yağıztegin Eroğlu <[email protected]>
linuxgemini added a commit to linuxgemini/acme-dns that referenced this pull request Feb 9, 2022
Co-authored-by: Matthias Schneider <[email protected]>
Co-authored-by: Johannes Doll <[email protected]>
Signed-off-by: İlteriş Yağıztegin Eroğlu <[email protected]>
t0xicCode added a commit to MagnaXSoftware/acme-dns that referenced this pull request Jun 16, 2022
t0xicCode added a commit to MagnaXSoftware/acme-dns that referenced this pull request Jul 24, 2023
t0xicCode added a commit to MagnaXSoftware/acme-dns that referenced this pull request Jul 24, 2023
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

Successfully merging this pull request may close these issues.

None yet

2 participants