make Prettier
This commit is contained in:
parent
8ab67b5a23
commit
844a668f70
86
index.js
86
index.js
|
@ -1,16 +1,16 @@
|
|||
"use strict";
|
||||
'use strict';
|
||||
/*global Promise*/
|
||||
var crypto = require("crypto");
|
||||
var crypto = require('crypto');
|
||||
|
||||
module.exports.create = function() {
|
||||
throw new Error(
|
||||
"acme-challenge-test is a test fixture for acme-challenge-* plugins, not a plugin itself"
|
||||
'acme-challenge-test is a test fixture for acme-challenge-* plugins, not a plugin itself'
|
||||
);
|
||||
};
|
||||
|
||||
// ignore all of this, it's just to normalize Promise vs node-style callback thunk vs synchronous
|
||||
function promiseCheckAndCatch(obj, name) {
|
||||
var promisify = require("util").promisify;
|
||||
var promisify = require('util').promisify;
|
||||
// don't loose this-ness, just in case that's important
|
||||
var fn = obj[name].bind(obj);
|
||||
var promiser;
|
||||
|
@ -32,13 +32,13 @@ function promiseCheckAndCatch(obj, name) {
|
|||
"'challenge." +
|
||||
name +
|
||||
"' should accept either one argument, the options," +
|
||||
" and return a Promise or accept two arguments, the options and a node-style callback thunk"
|
||||
' and return a Promise or accept two arguments, the options and a node-style callback thunk'
|
||||
)
|
||||
);
|
||||
}
|
||||
|
||||
function shouldntBeNull(result) {
|
||||
if ("undefined" === typeof result) {
|
||||
if ('undefined' === typeof result) {
|
||||
throw new Error(
|
||||
"'challenge.'" +
|
||||
name +
|
||||
|
@ -57,21 +57,21 @@ function promiseCheckAndCatch(obj, name) {
|
|||
// Here's the meat, where the tests are happening:
|
||||
function run(challenger, opts) {
|
||||
var ch = opts.challenge;
|
||||
if ("http-01" === ch.type && ch.wildname) {
|
||||
throw new Error("http-01 cannot be used for wildcard domains");
|
||||
if ('http-01' === ch.type && ch.wildname) {
|
||||
throw new Error('http-01 cannot be used for wildcard domains');
|
||||
}
|
||||
|
||||
var set = promiseCheckAndCatch(challenger, "set");
|
||||
if ("function" !== typeof challenger.get) {
|
||||
var set = promiseCheckAndCatch(challenger, 'set');
|
||||
if ('function' !== typeof challenger.get) {
|
||||
throw new Error(
|
||||
"'challenge.get' should be implemented for the sake of testing." +
|
||||
" It should be implemented as the internal method for fetching the challenge" +
|
||||
" (i.e. reading from a database, file system or API, not return internal)," +
|
||||
" not the external check (the http call, dns query, etc), which will already be done as part of this test."
|
||||
' It should be implemented as the internal method for fetching the challenge' +
|
||||
' (i.e. reading from a database, file system or API, not return internal),' +
|
||||
' not the external check (the http call, dns query, etc), which will already be done as part of this test.'
|
||||
);
|
||||
}
|
||||
var get = promiseCheckAndCatch(challenger, "get");
|
||||
var remove = promiseCheckAndCatch(challenger, "remove");
|
||||
var get = promiseCheckAndCatch(challenger, 'get');
|
||||
var remove = promiseCheckAndCatch(challenger, 'remove');
|
||||
|
||||
// The first time we just check it against itself
|
||||
// this will cause the prompt to appear
|
||||
|
@ -80,13 +80,13 @@ function run(challenger, opts) {
|
|||
// this will cause the final completion message to appear
|
||||
// _test is used by the manual cli reference implementations
|
||||
var query = { type: ch.type, /*debug*/ status: ch.status, _test: true };
|
||||
if ("http-01" === ch.type) {
|
||||
if ('http-01' === ch.type) {
|
||||
query.identifier = ch.identifier;
|
||||
query.token = ch.token;
|
||||
// For testing only
|
||||
query.url = ch.challengeUrl;
|
||||
} else if ("dns-01" === ch.type) {
|
||||
query.identifier = { type: "dns", value: ch.dnsHost };
|
||||
} else if ('dns-01' === ch.type) {
|
||||
query.identifier = { type: 'dns', value: ch.dnsHost };
|
||||
// For testing only
|
||||
query.altname = ch.altname;
|
||||
// there should only be two possible TXT records per challenge domain:
|
||||
|
@ -94,26 +94,26 @@ function run(challenger, opts) {
|
|||
query.wildcard = ch.wildcard;
|
||||
} else {
|
||||
query = JSON.parse(JSON.stringify(ch));
|
||||
query.comment = "unknown challenge type, supplying everything";
|
||||
query.comment = 'unknown challenge type, supplying everything';
|
||||
}
|
||||
return get({ challenge: query })
|
||||
.then(function(secret) {
|
||||
if ("string" === typeof secret) {
|
||||
if ('string' === typeof secret) {
|
||||
console.info(
|
||||
"secret was passed as a string, which works historically, but should be an object instead:"
|
||||
'secret was passed as a string, which works historically, but should be an object instead:'
|
||||
);
|
||||
console.info('{ "keyAuthorization": "' + secret + '" }');
|
||||
console.info("or");
|
||||
console.info('or');
|
||||
// TODO this should be "keyAuthorizationDigest"
|
||||
console.info('{ "dnsAuthorization": "' + secret + '" }');
|
||||
console.info(
|
||||
"This is to help keep acme / greenlock (and associated plugins) future-proof for new challenge types"
|
||||
'This is to help keep acme / greenlock (and associated plugins) future-proof for new challenge types'
|
||||
);
|
||||
}
|
||||
// historically 'secret' has been a string, but I'd like it to transition to be an object.
|
||||
// to make it backwards compatible in v2.7 to change it,
|
||||
// so I'm not sure that we really need to.
|
||||
if ("http-01" === ch.type) {
|
||||
if ('http-01' === ch.type) {
|
||||
secret = secret.keyAuthorization || secret;
|
||||
if (ch.keyAuthorization !== secret) {
|
||||
throw new Error(
|
||||
|
@ -125,7 +125,7 @@ function run(challenger, opts) {
|
|||
"'"
|
||||
);
|
||||
}
|
||||
} else if ("dns-01" === ch.type) {
|
||||
} else if ('dns-01' === ch.type) {
|
||||
secret = secret.dnsAuthorization || secret;
|
||||
if (ch.dnsAuthorization !== secret) {
|
||||
throw new Error(
|
||||
|
@ -138,7 +138,7 @@ function run(challenger, opts) {
|
|||
);
|
||||
}
|
||||
} else {
|
||||
if ("tls-alpn-01" === ch.type) {
|
||||
if ('tls-alpn-01' === ch.type) {
|
||||
console.warn(
|
||||
"'tls-alpn-01' support is in development" +
|
||||
" (or developed and we haven't update this yet). Please contact us."
|
||||
|
@ -165,12 +165,12 @@ function run(challenger, opts) {
|
|||
return get(opts).then(function(result) {
|
||||
if (result) {
|
||||
throw new Error(
|
||||
"challenge.remove() should have made it not possible for challenge.get() to return a value"
|
||||
'challenge.remove() should have made it not possible for challenge.get() to return a value'
|
||||
);
|
||||
}
|
||||
if (null !== result) {
|
||||
throw new Error(
|
||||
"challenge.get() should return null when the value is not set"
|
||||
'challenge.get() should return null when the value is not set'
|
||||
);
|
||||
}
|
||||
});
|
||||
|
@ -178,9 +178,9 @@ function run(challenger, opts) {
|
|||
});
|
||||
})
|
||||
.then(function() {
|
||||
console.info("All soft tests: PASS");
|
||||
console.info('All soft tests: PASS');
|
||||
console.warn(
|
||||
"Hard tests (actually checking http URLs and dns records) is implemented in acme-v2."
|
||||
'Hard tests (actually checking http URLs and dns records) is implemented in acme-v2.'
|
||||
);
|
||||
console.warn(
|
||||
"We'll copy them over here as well, but that's a TODO for next week."
|
||||
|
@ -190,39 +190,39 @@ function run(challenger, opts) {
|
|||
|
||||
module.exports.test = function(type, altname, challenger) {
|
||||
var expires = new Date(Date.now() + 10 * 60 * 1000).toISOString();
|
||||
var token = crypto.randomBytes(8).toString("hex");
|
||||
var thumb = crypto.randomBytes(16).toString("hex");
|
||||
var keyAuth = token + "." + crypto.randomBytes(16).toString("hex");
|
||||
var token = crypto.randomBytes(8).toString('hex');
|
||||
var thumb = crypto.randomBytes(16).toString('hex');
|
||||
var keyAuth = token + '.' + crypto.randomBytes(16).toString('hex');
|
||||
var dnsAuth = crypto
|
||||
.createHash("sha256")
|
||||
.createHash('sha256')
|
||||
.update(keyAuth)
|
||||
.digest("base64")
|
||||
.replace(/\+/g, "-")
|
||||
.replace(/\//g, "_")
|
||||
.replace(/=/g, "");
|
||||
.digest('base64')
|
||||
.replace(/\+/g, '-')
|
||||
.replace(/\//g, '_')
|
||||
.replace(/=/g, '');
|
||||
|
||||
var challenge = {
|
||||
type: type,
|
||||
identifier: { type: "dns", value: null }, // completed below
|
||||
identifier: { type: 'dns', value: null }, // completed below
|
||||
wildcard: false, // completed below
|
||||
status: "pending",
|
||||
status: 'pending',
|
||||
expires: expires,
|
||||
token: token,
|
||||
thumbprint: thumb,
|
||||
keyAuthorization: keyAuth,
|
||||
url: null, // completed below
|
||||
dnsHost: "_acme-challenge.", // completed below
|
||||
dnsHost: '_acme-challenge.', // completed below
|
||||
dnsAuthorization: dnsAuth,
|
||||
altname: altname,
|
||||
_test: true // used by CLI referenced implementations
|
||||
};
|
||||
if ("*." === altname.slice(0, 2)) {
|
||||
if ('*.' === altname.slice(0, 2)) {
|
||||
challenge.wildcard = true;
|
||||
altname = altname.slice(2);
|
||||
}
|
||||
challenge.identifier.value = altname;
|
||||
challenge.url =
|
||||
"http://" + altname + "/.well-known/acme-challenge/" + challenge.token;
|
||||
'http://' + altname + '/.well-known/acme-challenge/' + challenge.token;
|
||||
challenge.dnsHost += altname;
|
||||
|
||||
return run(challenger, { challenge: challenge });
|
||||
|
|
Loading…
Reference in New Issue