This repository has been archived by the owner on Apr 26, 2024. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Support ACME for certificate provisioning #4384
Support ACME for certificate provisioning #4384
Changes from 40 commits
7273a5b
bebd71e
17a161e
35e7768
895ef68
75ec773
bad5b23
727bfc2
c5fbe94
b999a09
9461c00
ddc7a17
d9bf3eb
04f805b
aefd46e
ac3ca16
4740f7d
603f896
bb7861a
1e9c0ad
4ab63ea
f1b9b0f
840fcb0
b616b89
248d943
f1dfb47
bd8114c
38b86cb
6db5717
940271c
723f3a1
fa9b705
8b92348
52d71ec
05ac907
f449d20
b0ff1ca
7f72038
54350ea
26787bd
e01e824
d601f85
0cc093f
9051fd8
64e5b41
2f5e68c
6d16053
349ab14
f150ec6
956f72d
521bc24
08ebd2e
a7f6727
77eef86
b58e684
fe15602
7323166
02a2a49
c12ba8e
8a44d61
67bccf7
1fb1f9c
30af4ad
947ab0a
57ec9f3
4db63ab
c293b21
392cfe9
380b01d
1a70173
0baf9ee
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't forget to set the default to be the live endpoint
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also, do we want to consider using the v2 API now , if it's a simple change in our use of the library, to prevent us having to move to v2 when they decommission (timeline unspecified, so if it's a lot of work it is definitelyworth punting down the road)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
txacme does not yet support v2. We can probably put a little bit of work in to make it support it, later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
WRT live endpoint -- I think it should be the staging one by default, since the real one has rate limits, and we dont' want someone to get accidentally blacklisted while setting up their server, I think.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that feels like a lower risk then them wondering why their cert doesn't work, tbh. The fact that the whole thing is disabled by default is enough of a safetynet imho.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm, okay