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

Acme v2 Support #555

Merged
merged 22 commits into from
May 30, 2018
Merged

Acme v2 Support #555

merged 22 commits into from
May 30, 2018

Conversation

ldez
Copy link
Member

@ldez ldez commented May 30, 2018

Merge the branch acmev2 into master.

Closes #457

xenolf and others added 22 commits March 14, 2018 02:04
* ResolveAccountByKey no longer returns an EOF error on success.

* Added test for ResolveAccountByKey.
# Conflicts:
#	providers/dns/azure/azure.go
- Switched out some value -> pointer returns
  for functions that returned an error
- Switched out previous failures map with an
  error
* Second draft of External Account Binding support with xenolf's proposed
changes included.
* Require --eab if the ACME directory says it requires External Account
Binding.
* Inner EAB JWS should not contain any nonce. Ref: https://ietf-wg-acme.github.io/acme/draft-ietf-acme-acme.html#rfc.section.7.3.5
- remove go1.8, due to miekg/dns#683
- add go1.x
Copy link
Contributor

@mholt mholt left a comment

Choose a reason for hiding this comment

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

Been using the v2 branch for months in production. Needs a little work but nothing major AFAICT. Merge away!

@ldez ldez merged commit e7fd871 into master May 30, 2018
@ldez ldez deleted the feature/merge-acmev2 branch May 30, 2018 17:53
@ldez ldez changed the title Merge acmev2 into master Acme v2 Support May 30, 2018
@ldez ldez added the area/lib label May 30, 2018
@ldez ldez modified the milestones: v0.1.0, v1.0 Dec 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

7 participants