Skip LE challenge verification. (#2280)

The verification was implemented wrong and could cause the registration process to hang for 20+ minutes. A newer, unreleased version of acme-client has an option to skip the verification step, so use that.
WebThingsIO · Nov 12, 2019 · fec6a9b · fec6a9b
1 parent c24eac2
commit fec6a9b
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -32,7 +32,7 @@
   "dependencies": {
     "@fluent/bundle": "^0.14.0",
     "@fluent/dom": "^0.6.0",
-    "acme-client": "^3.2.0",
+    "acme-client": "https://github.com/publishlab/node-acme-client#7bb000d561cda8e31e3dc82e6f499d947dd92a0e",
     "ajv": "^6.10.2",
     "archiver": "^3.1.1",
     "asn1.js": "^5.2.0",

diff --git a/src/certificate-manager.js b/src/certificate-manager.js
@@ -183,6 +183,7 @@ async function register(email, reclamationToken, subdomain, fulldomain,
       csr,
       email: config.get('ssltunnel.certemail'),
       termsOfServiceAgreed: true,
+      skipChallengeVerification: true,
       challengePriority: ['dns-01'],
       challengeCreateFn,
       challengeRemoveFn,