Add missing field to regr.json. #1

Merged
coolaj86 merged 1 commits from :new-authzr-uri into master 2018-05-21 17:13:38 +00:00
First-time contributor

When using generated configs with older certbot versions,
i.e. <0.11, certbot bombs out without new_authzr_uri.

When using generated configs with older certbot versions, i.e. <0.11, certbot bombs out without new_authzr_uri.
Owner

We now use ACME draft-11 / Let's Encrypt v02 by default, not v01.

We also don't want to assume and hard-code the use of the Let's Encrypt brand ACME provider.

I think that right around here https://git.coolaj86.com/coolaj86/greenlock.js/src/branch/master/lib/core.js we need to add args._acmeUrls.newReg as a parameter and then use that instead of the hard-coded url.

Will you test that and make a PR? I'll try to publish today if you can make the change, confirm that that works with v01 and certbot, and I can confirm that it works with v02 without certbot.

We now use ACME draft-11 / Let's Encrypt v02 by default, not v01. We also don't want to assume and hard-code the use of the Let's Encrypt brand ACME provider. I think that right around here https://git.coolaj86.com/coolaj86/greenlock.js/src/branch/master/lib/core.js we need to add `args._acmeUrls.newReg` as a parameter and then use that instead of the hard-coded url. Will you test that and make a PR? I'll try to publish today if you can make the change, confirm that that works with v01 and certbot, and I can confirm that it works with v02 without certbot.
Author
First-time contributor

Was there supposed to be a line number in there?

Was there supposed to be a line number in there?
Owner

Yes... there was.

I think I was referencing Lines 92 and 109

https://git.coolaj86.com/coolaj86/greenlock.js/src/branch/master/lib/core.js#L109

Yes... there was. I think I was referencing Lines 92 and 109 https://git.coolaj86.com/coolaj86/greenlock.js/src/branch/master/lib/core.js#L109
Author
First-time contributor

Finally got around to this.

greenlock change: coolaj86/greenlock.js#13

I also updated this PR to use that field.

Finally got around to this. greenlock change: https://git.coolaj86.com/coolaj86/greenlock.js/pulls/13 I also updated this PR to use that field.
coolaj86 closed this pull request 2018-05-21 17:13:38 +00:00
Owner

Thanks much.

Merged and published as v2.1.2.

Now going to check out the other...

Thanks much. Merged and published as v2.1.2. Now going to check out the other...
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: coolaj86/le-store-certbot.js#1
No description provided.