.prettierrc

@@ -1,8 +1,4 @@
 {
-  "bracketSpacing": true,
-  "printWidth": 80,
-  "singleQuote": true,
-  "tabWidth": 2,
   "trailingComma": "none",
-  "useTabs": false
+  "useTabs": true
 }

README.md

@@ -1,70 +1,98 @@
-# [acme-challenge-test](https://git.rootprojects.org/root/acme-challenge-test.js.git) | a [Root](https://rootprojects.org) project
+# [acme-challenge-test](https://git.rootprojects.org/root/acme-challenge-test.js.git) | A [Root](https://rootprojects.org) Project
 
-Test harness for ACME http-01 and dns-01 challenges for Let's Encrypt Free SSL integration.
+The test harness you should use when writing an ACME challenge strategy
+for [Greenlock](https://git.coolaj86.com/coolaj86/greenlock-express.js) v2.7+ (and v3).
 
-This was specificially designed for [ACME.js](https://git.coolaj86.com/coolaj86/acme-v2.js) and [Greenlock.js](https://git.coolaj86.com/coolaj86/greenlock-express.js), but will be generically useful to any ACME module.
+All implementations MUST pass these tests, which is a very easy thing to do (just `set()`, `get()`, and `remove()`).
 
-> If you are building a plugin, please let us know.
+The tests account for single-domain certificates (`example.com`) as well as multiple domain certs (SAN / AltName),
-> We may like to co-author and help maintain and promote your module.
+wildcards (`*.example.com`), and valid private / localhost certificates. As someone creating a challenge strategy
+that's not something you have to take special consideration for - just pass the tests.
 
-This package has been split in two for the purpose of keeping the documentation clear and concise.
+## Install
 
-## ACME http-01
+```bash
+npm install --save-dev acme-challenge-test@3.x
+```
 
-Use this for quick-and-easy, average-joe kind of stuff.
+## Usage
 
-- See <https://git.rootprojects.org/root/acme-http-01-test.js>
+```js
+var tester = require("acme-challenge-test");
 
-## ACME dns-01
+//var challenger = require('acme-http-01-cli').create({});
+//var challenger = require('acme-dns-01-cli').create({});
+var challenger = require("./YOUR-CHALLENGE-STRATEGY").create({});
 
-Use this for wildcards, and private and local domains.
+// The dry-run tests can pass on, literally, 'example.com'
+// but the integration tests require that you have control over the domain
+var domain = "example.com";
 
-- See <https://git.rootprojects.org/root/acme-dns-01-test.js>
+tester.test("http-01", domain, challenger).then(function() {
+	console.info("PASS");
+});
+```
 
 ## Reference Implementations
 
 These are plugins that use the v2.7+ (v3) API, and pass this test harness,
 which you should use as a model for any plugins that you create.
 
-- http-01
+- [`acme-http-01-cli`](https://git.rootprojects.org/root/acme-http-01-cli.js)
-  - [`acme-http-01-cli`](https://git.rootprojects.org/root/acme-http-01-cli.js)
+- [`acme-dns-01-cli`](https://git.rootprojects.org/root/acme-dns-01-cli.js)
-  - [`acme-http-01-fs`](https://git.rootprojects.org/root/acme-http-01-fs.js)
-- dns-01
-  - [`acme-dns-01-cli`](https://git.rootprojects.org/root/acme-dns-01-cli.js)
-  - [`acme-dns-01-digitalocean`](https://git.rootprojects.org/root/acme-dns-01-digitalocean.js)
 
-You can find other implementations by searching npm for [acme-http-01-](https://www.npmjs.com/search?q=acme-http-01-)
+## Example
-and [acme-dns-01-](https://www.npmjs.com/search?q=acme-dns-01-).
 
-## Starter Template
+See `example.js` (it works).
 
-Just so you have an idea, this is typically how you'd start passing the tests:
+## Overview
+
+Here's a quick pseudo stub-out of what a test-passing plugin object might look like:
 
 ```js
-var tester = require('acme-challenge-test');
+tester.test('http-01', 'example.com', {
+  set: function (opts) {
+    var ch = opts.challenge;
+    // { type: 'http-01' // or 'dns-01'
+    // , identifier: { type: 'dns', value: 'example.com' }
+    // , wildcard: false
+    // , token: 'xxxx'
+    // , keyAuthorization: 'xxxx.yyyy'
+    // , dnsHost: '_acme-challenge.example.com'
+    // , dnsAuthorization: 'zzzz' }
 
-// The dry-run tests can pass on, literally, 'example.com'
+    return API.set(...);
-// but the integration tests require that you have control over the domain
-var domain = 'example.com';
-
-tester
-  .testRecord('http-01', domain, {
-    set: function(opts) {
-      console.log('set opts:', opts);
-      throw new Error('set not implemented');
-    },
-
-    remove: function(opts) {
-      console.log('remove opts:', opts);
-      throw new Error('remove not implemented');
-    },
-
-    get: function(opts) {
-      console.log('get opts:', opts);
-      throw new Error('get not implemented');
   }
-  })
+, get: function (query) {
-  .then(function() {
+    var ch = query.challenge;
-    console.info('PASS');
+    // { type: 'http-01' // or 'dns-01', 'tls-alpn-01', etc
+    // , identifier: { type: 'dns', value: 'example.com' }
+    //   // http-01 only
+    // , token: 'xxxx'
+    // , url: '...' // for testing and debugging
+    //   // dns-01 only, for testing / dubgging
+    // , altname: '...'
+    // , dnsHost: '...'
+    // , wildcard: false }
+    // Note: query.identifier.value is different for http-01 than for dns-01
+
+    return API.get(...).then(function (secret) {
+      // http-01
+      return { keyAuthorization: secret };
+      // dns-01
+      //return { dnsAuthorization: secret };
     });
+  }
+, remove: function (opts) {
+    var ch = opts.challenge;
+    // same options as in `set()` (which are not the same as `get()`
+
+    return API.remove(...);
+  }
+}).then(function () {
+  console.info("PASS");
+});
 ```
+
+Note: The `API.get()`, `API.set()`, and `API.remove()` is where you do your magic up to upload a file to the correct
+location on an http serever, set DNS records, or add the appropriate data to the database that handles such things.

example.js

@@ -1,26 +1,27 @@
-'use strict';
+"use strict";
 
 //var tester = require('acme-challenge-test');
-var tester = require('./');
+var tester = require("./");
 
-//var type = 'http-01';
+var type = "http-01";
-//var challenger = require('acme-http-01-cli').create({});
+var challenger = require("acme-http-01-cli").create({});
-var type = 'dns-01';
+//var type = 'dns-01';
-var challenger = require('acme-dns-01-cli').create({});
+//var challenger = require('acme-dns-01-cli').create({});
 //var challenger = require('./YOUR-CHALLENGE-STRATEGY').create({});
 //var type = 'YOUR-TYPE-01';
 
 // The dry-run tests can pass on, literally, 'example.com'
 // but the integration tests require that you have control over the domain
-var zone = 'example.com';
+var domain = "example.com";
+//var domain = '*.example.com';
 
 tester
-  .test(type, zone, challenger)
+	.test(type, domain, challenger)
 	.then(function() {
-    console.info('ALL PASSED');
+		console.info("PASS");
 	})
 	.catch(function(err) {
-    console.error('FAIL');
+		console.error("FAIL");
 		console.error(err);
 		process.exit(20);
 	});

index.js

@@ -1,44 +1,22 @@
-'use strict';
+"use strict";
 /*global Promise*/
-
+var crypto = require("crypto");
-if (process.version.match(/^v(\d+)/)[1] > 6) {
-  console.warn();
-  console.warn('#########################');
-  console.warn('# Node v6 Compatibility #');
-  console.warn('#########################');
-  console.warn();
-  console.warn("You're using node " + process.version);
-  console.warn(
-    'Please write node-v6 compatible JavaScript (not Babel/ECMAScript) and test with node v6.'
-  );
-  console.warn();
-  console.warn(
-    '(ACME.js and Greenlock.js are widely deployed in enterprise node v6 environments. The few node v6 bugs in Buffer and Promise are hotfixed by ACME.js in just a few lines of code)'
-  );
-  console.warn();
-}
-require('./lib/node-v6-compat.js');
-
-// Load _after_ node v6 compat
-var crypto = require('crypto');
-var promisify = require('util').promisify;
-var request = require('@root/request');
-request = promisify(request);
 
 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;
 	// don't loose this-ness, just in case that's important
 	var fn = obj[name].bind(obj);
 	var promiser;
 
 	// function signature must match, or an error will be thrown
-  if (fn.length <= 1) {
+	if (1 === fn.length) {
 		// wrap so that synchronous errors are caught (alsa handles synchronous results)
 		promiser = function(opts) {
 			return Promise.resolve().then(function() {
@@ -49,20 +27,22 @@ function promiseCheckAndCatch(obj, name) {
 		// wrap as a promise
 		promiser = promisify(fn);
 	} else {
-    throw new Error(
+		return Promise.reject(
+			new Error(
 				"'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 shouldntBeUndefined(result) {
+	function shouldntBeNull(result) {
-    if ('undefined' === typeof result) {
+		if ("undefined" === typeof result) {
 			throw new Error(
 				"'challenge.'" +
 					name +
-          "' should never return `undefined`. Please explicitly `return null`" +
+					"' should never return `undefined`. Please explicitly return null" +
 					" (or fix the place where a value should have been returned but wasn't)."
 			);
 		}
@@ -70,179 +50,70 @@ function promiseCheckAndCatch(obj, name) {
 	}
 
 	return function(opts) {
-    return promiser(opts).then(shouldntBeUndefined);
+		return promiser(opts).then(shouldntBeNull);
 	};
 }
 
-function mapAsync(els, doer) {
-  els = els.slice(0);
-  var results = [];
-  function next() {
-    var el = els.shift();
-    if (!el) {
-      return results;
-    }
-    return doer(el).then(function(result) {
-      results.push(result);
-      return next();
-    });
-  }
-  return next();
-}
-
-function newZoneRegExp(zonename) {
-  // (^|\.)example\.com$
-  // which matches:
-  //  foo.example.com
-  //  example.com
-  // but not:
-  //  fooexample.com
-  return new RegExp('(^|\\.)' + zonename.replace(/\./g, '\\.') + '$');
-}
-
-function pluckZone(zones, dnsHost) {
-  return zones
-    .filter(function(zonename) {
-      // the only character that needs to be escaped for regex
-      // and is allowed in a domain name is '.'
-      return newZoneRegExp(zonename).test(dnsHost);
-    })
-    .sort(function(a, b) {
-      // longest match first
-      return b.length - a.length;
-    })[0];
-}
-
 // Here's the meat, where the tests are happening:
-function testEach(type, domains, challenger) {
+function run(challenger, opts) {
-  var chr = wrapChallenger(type, challenger);
-  // We want the same rnd for all domains so that we catch errors like removing
-  // the apex (bare) domain TXT record to when creating the wildcard record
-  var rnd = crypto.randomBytes(2).toString('hex');
-
-  console.info("Testing each of '%s'", domains.join(', '));
-
-  //
-  // Zones
-  //
-  // Get ALL zones for all records on the certificate
-  //
-  return chr
-    .init({ request: request })
-    .then(function() {
-      return chr.zones({ request: request, dnsHosts: domains });
-    })
-    .then(function(zones) {
-      var all = domains.map(function(domain) {
-        var zone = pluckZone(zones, domain);
-        return {
-          domain: domain,
-          challenge: fakeChallenge(type, zone, domain, rnd),
-          request: request
-        };
-      });
-
-      // resolving for the sake of same indentation / scope
-      return Promise.resolve()
-        .then(function() {
-          return mapAsync(all, function(opts) {
-            return set(chr, opts);
-          });
-        })
-        .then(function() {
-          return mapAsync(all, function(opts) {
-            return check(chr, opts);
-          });
-        })
-        .then(function() {
-          return mapAsync(all, function(opts) {
-            return remove(chr, opts).then(function() {
-              console.info("PASS '%s'", opts.domain);
-            });
-          });
-        })
-        .then(function() {
-          console.info();
-          console.info('It looks like the soft tests all passed.');
-          console.log('It is highly likely that your plugin is correct.');
-          console.log(
-            'Now go test with Greenlock.js and/or ACME.js to be sure.'
-          );
-          console.info();
-        });
-    });
-}
-
-function set(chr, opts) {
 	var ch = opts.challenge;
-  if ('http-01' === ch.type && ch.wildname) {
+	if ("http-01" === ch.type && ch.wildname) {
-    throw new Error('http-01 cannot be used for wildcard domains');
+		throw new Error("http-01 cannot be used for wildcard domains");
 	}
 
-  //
+	var set = promiseCheckAndCatch(challenger, "set");
-  // Set
+	if ("function" !== typeof challenger.get) {
-  //
+		throw new Error(
-  // Add (not replace) a TXT for the domain
+			"'challenge.get' should be implemented for the sake of testing." +
-  //
+				" It should be implemented as the internal method for fetching the challenge" +
-  return chr.set(opts).then(function() {
+				" (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");
+
+	// The first time we just check it against itself
+	// this will cause the prompt to appear
+	return set(opts)
+		.then(function() {
+			// 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) {
+			} else if ("dns-01" === ch.type) {
-      query.identifier = { type: 'dns', value: ch.dnsHost };
+				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:
 				// one for the bare domain, and the other if and only if there's a wildcard
 				query.wildcard = ch.wildcard;
-      query.dnsAuthorization = ch.dnsAuthorization;
-      query.dnsZone = ch.dnsZone;
-      query.dnsPrefix = ch.dnsPrefix;
 			} else {
 				query = JSON.parse(JSON.stringify(ch));
-      query.comment = 'unknown challenge type, supplying everything';
+				query.comment = "unknown challenge type, supplying everything";
 			}
-    opts.query = query;
+			return get({ challenge: query })
-    return opts;
-  });
-}
-
-function check(chr, opts) {
-  var ch = opts.challenge;
-
-  //
-  // Get
-  //
-  // Check that ONE of the relevant TXT records matches
-  //
-  return chr
-    .get({ request: request, challenge: opts.query })
 				.then(function(secret) {
-      if (!secret) {
+					if ("string" === typeof secret) {
-        throw new Error(
-          '`secret` should be an object containing `keyAuthorization` or `dnsAuthorization`'
-        );
-      }
-      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(
@@ -254,7 +125,7 @@ function check(chr, opts) {
 									"'"
 							);
 						}
-      } else if ('dns-01' === ch.type) {
+					} else if ("dns-01" === ch.type) {
 						secret = secret.dnsAuthorization || secret;
 						if (ch.dnsAuthorization !== secret) {
 							throw new Error(
@@ -267,7 +138,7 @@ function check(chr, 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."
@@ -288,137 +159,71 @@ function check(chr, opts) {
 							);
 						}
 					}
-    });
+				})
-}
+				.then(function() {
-
+					return remove(opts).then(function() {
-function remove(chr, opts) {
+						return get(opts).then(function(result) {
-  //
-  // Remove
-  //
-  // Delete ONLY the SINGLE relevant TXT record
-  //
-  return chr.remove(opts).then(function() {
-    return chr.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"
 								);
 							}
 						});
 					});
+				});
+		})
+		.then(function() {
+			console.info("All soft tests: PASS");
+			console.warn(
+				"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."
+			);
+		});
 }
 
-function wrapChallenger(type, challenger) {
+module.exports.test = function(type, altname, challenger) {
-  var zones;
-  if ('dns-01' === type) {
-    if ('function' !== typeof challenger.zones) {
-      console.error(
-        'You must implement `zones` to return an array of strings.' +
-          " If you're testing a special type of service that doesn't support" +
-          ' domain zone listing (as opposed to domain record listing),' +
-          ' such as DuckDNS, return an empty array.'
-      );
-      process.exit(28);
-      return;
-    }
-    zones = promiseCheckAndCatch(challenger, 'zones');
-  } else {
-    zones = function() {
-      return Promise.resolve([]);
-    };
-  }
-
-  if ('function' !== typeof challenger.get) {
-    console.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.'
-    );
-    process.exit(29);
-    return;
-  }
-
-  var init = challenger.init;
-  if ('function' !== typeof init) {
-    init = function(opts) {
-      return null;
-    };
-  }
-  return {
-    init: promiseCheckAndCatch(challenger, 'init'),
-    zones: zones,
-    set: promiseCheckAndCatch(challenger, 'set'),
-    get: promiseCheckAndCatch(challenger, 'get'),
-    remove: promiseCheckAndCatch(challenger, 'remove')
-  };
-}
-
-function fakeChallenge(type, zone, altname, rnd) {
 	var expires = new Date(Date.now() + 10 * 60 * 1000).toISOString();
-  var token = crypto.randomBytes(8).toString('hex');
+	var token = crypto.randomBytes(8).toString("hex");
-  var thumb = crypto.randomBytes(16).toString('hex');
+	var thumb = crypto.randomBytes(16).toString("hex");
-  var keyAuth = token + '.' + crypto.randomBytes(16).toString('hex');
+	var keyAuth = token + "." + crypto.randomBytes(16).toString("hex");
 	var dnsAuth = crypto
-    .createHash('sha256')
+		.createHash("sha256")
 		.update(keyAuth)
-    .digest('base64')
+		.digest("base64")
-    .replace(/\+/g, '-')
+		.replace(/\+/g, "-")
-    .replace(/\//g, '_')
+		.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
-    // we create a random record to prevent self cache-poisoning
+		dnsHost: "_acme-challenge.", // completed below
-    dnsHost: '_' + rnd.slice(0, 2) + '-acme-challenge-' + rnd.slice(2) + '.', // 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;
-  if (zone) {
-    challenge.dnsZone = zone;
-    challenge.dnsPrefix = challenge.dnsHost
-      .replace(newZoneRegExp(zone), '')
-      .replace(/\.$/, '');
-  }
 
-  return challenge;
+	return run(challenger, { challenge: challenge });
-}
+};
-
-function testZone(type, zone, challenger) {
-  var domains = [zone, 'foo.' + zone];
-  if ('dns-01' === type) {
-    domains.push('*.foo.' + zone);
-  }
-  return testEach(type, domains, challenger);
-}
-
-function testRecord(type, altname, challenger) {
-  return testEach(type, [altname], challenger);
-}
-
-module.exports.testRecord = testRecord;
-module.exports.testZone = testZone;
-module.exports.test = testZone;

lib/node-v6-compat.js

@@ -1,25 +0,0 @@
-"use strict";
-
-function requireBluebird() {
-	try {
-		return require("bluebird");
-	} catch (e) {
-		console.error("");
-		console.error("DON'T PANIC. You're running an old version of node with incomplete Promise support.");
-		console.error("EASY FIX: `npm install --save bluebird`");
-		console.error("");
-		throw e;
-	}
-}
-
-if ("undefined" === typeof Promise) {
-	global.Promise = requireBluebird();
-}
-
-if ("function" !== typeof require("util").promisify) {
-	require("util").promisify = requireBluebird().promisify;
-}
-
-if (!console.debug) {
-	console.debug = console.log;
-}

package.json

@@ -1,16 +1,10 @@
 {
 	"name": "acme-challenge-test",
-  "version": "3.3.2",
+	"version": "3.0.3",
-  "description": "ACME challenge test harness for Let's Encrypt integrations. Any `acme-http-01-` or `acme-dns-01-` challenge strategy or Greenlock plugin should be able to pass these tests.",
+	"description": "The base set of tests for all ACME challenge strategies. Any `acme-http-01-`, `acme-dns-01-`, `acme-challenge-`, or greenlock plugin should be able to pass these tests.",
 	"main": "index.js",
 	"homepage": "https://git.rootprojects.org/root/acme-challenge-test.js",
-  "files": [
+	"dependencies": {},
-    "example.js",
-    "lib"
-  ],
-  "dependencies": {
-    "@root/request": "^1.3.11"
-  },
 	"devDependencies": {},
 	"scripts": {
 		"test": "node example.js"
@@ -27,8 +21,7 @@
 		"challenge",
 		"plugin",
 		"module",
-    "strategy",
+		"strategy"
-    "greenlock"
 	],
 	"author": "AJ ONeal <solderjs@gmail.com> (https://solderjs.com/)",
 	"license": "MPL-2.0"