.gitignore

@@ -1,5 +1,3 @@
-*delete-me*
-
 # ---> Node
 # Logs
 logs

README.md

@@ -10,28 +10,22 @@ A simple file-based management strategy for Greenlock v3
 npm install --save greenlock-manager-fs@v3
 ```
 
-## Usage
+## Use with Greenlock
 
 ```js
-npx greenlock init --manager greenlock-manager-fs --config-dir './greenlock.d'
+var greenlock = require('greenlock').create({
-```
+	// ...
 
-Or, place this file in the root of your project:
+	manager: 'greenlock-manager-fs',
-
+	configFile: '~/.config/greenlock/manager.json'
-`.greenlockrc`:
+});
-
-```json
-{
-	"manager": { "module": "@greenlock/manager" },
-	"configDir": "./greenlock.d"
-}
 ```
 
 ## Example config file
 
 You might start your config file like this:
 
-`./greenlock.d/config.json`:
+`~/.config/greenlock/manager.json`:
 
 ```json
 {

manager.js

@@ -19,13 +19,8 @@ var mkdirp = promisify(require('@root/mkdirp'));
 // and wanting to be fairly sure it works and produces
 // meaningful errors
 
-// IMPORTANT
 // For your use case you'll probably find a better example
-// in greenlock-manager-test:
+// in greenlock-manager-test
-//
-//    npm install --save greenlock-manager-test
-//    npx greenlock-manager-init
-//
 
 Manage.create = function(CONF) {
 	if (!CONF) {
@@ -80,31 +75,6 @@ Manage.create = function(CONF) {
 		return manage._txPromise;
 	};
 
-	manage.get = async function(args) {
-		manage._txPromise = manage._txPromise.then(async function() {
-			var config = await Manage._getLatest(manage, CONF);
-			var site;
-			Object.keys(config.sites).some(function(k) {
-				// if subject is specified, don't return anything else
-				var _site = config.sites[k];
-
-				// altnames, servername, and wildname all get rolled into one
-				return _site.altnames.some(function(altname) {
-					if ([args.servername, args.wildname].includes(altname)) {
-						site = _site;
-					}
-				});
-			});
-
-			if (site && !site.deletedAt) {
-				return doctor.site(config.sites, site.subject);
-			}
-			return null;
-		});
-
-		return manage._txPromise;
-	};
-
 	manage._merge = function(config, current, args) {
 		if (!current || current.deletedAt) {
 			current = config.sites[args.subject] = {
@@ -125,15 +95,10 @@ Manage.create = function(CONF) {
 			if (newAlts.join() !== oldAlts.join()) {
 				// this will cause immediate renewal
 				args.renewAt = 1;
+				current.renewAt = 1;
 				current.altnames = args.altnames.slice(0);
 			}
 		}
-		Object.keys(args).forEach(function(k) {
-			if ('altnames' === k) {
-				return;
-			}
-			current[k] = args[k];
-		});
 	};
 
 	// no transaction promise here because it calls set
@@ -147,8 +112,7 @@ Manage.create = function(CONF) {
 		// between the managed state and the overlay, and choose
 		// what was found.
 		var theirs = await CONF.find(args);
-		var config = await Manage._getLatest(manage, CONF);
+		return _mergeFind(ours, theirs);
-		return _mergeFind(config, ours, theirs);
 	};
 
 	function _find(args) {
@@ -214,7 +178,7 @@ Manage.create = function(CONF) {
 
 	function _mergeFind(config, ours, theirs) {
 		theirs.forEach(function(_newer) {
-			var hasCurrent = ours.some(function(_older) {
+			ours.some(function(_older) {
 				if (_newer.subject !== _older.subject) {
 					return false;
 				}
@@ -228,9 +192,6 @@ Manage.create = function(CONF) {
 				// handled the (only) match
 				return true;
 			});
-			if (hasCurrent) {
-				manage._merge(config, null, _newer);
-			}
 		});
 
 		// delete the things that are gone
@@ -247,10 +208,7 @@ Manage.create = function(CONF) {
 			await manage._save(config);
 			// everything was either added, updated, or not different
 			// hence, this is everything
-			var copy = JSON.parse(JSON.stringify(config.sites));
+			return JSON.parse(JSON.stringify(config.sites));
-			return Object.keys(copy).map(function(k) {
-				return copy[k];
-			});
 		});
 
 		return manage._txPromise;
@@ -263,7 +221,7 @@ Manage.create = function(CONF) {
 		manage._txPromise = manage._txPromise.then(async function() {
 			var config = await Manage._getLatest(manage, CONF);
 			var site = config.sites[args.subject];
-			if (!site || site.deletedAt) {
+			if (!site) {
 				return null;
 			}
 			site.deletedAt = Date.now();
@@ -280,16 +238,8 @@ Manage.create = function(CONF) {
 	manage._save = async function(config) {
 		await mkdirp(path.dirname(CONF.configFile));
 		// pretty-print the config file
-		var data = JSON.parse(JSON.stringify(config));
+		var data = JSON.stringify(config, null, 2);
-		var sites = data.sites || {};
+		await sfs.writeFileAsync(CONF.configFile, data, 'utf8');
-		data.sites = Object.keys(sites).map(function(k) {
-			return sites[k];
-		});
-		await sfs.writeFileAsync(
-			CONF.configFile,
-			JSON.stringify(data, null, 2),
-			'utf8'
-		);
 
 		// this file may contain secrets, so keep it safe
 		return chmodFile(CONF.configFile, parseInt('0600', 8))
@@ -304,9 +254,8 @@ Manage.create = function(CONF) {
 	};
 
 	manage.init = async function(deps) {
-		// even though we don't need it
+		var request = deps.request;
-		manage.request = deps.request;
+		// how nice...
-		return null;
 	};
 
 	return manage;
@@ -349,10 +298,7 @@ doctor.config = function(config) {
 	doctor.sites(config);
 
 	Object.keys(config).forEach(function(key) {
-		// .greenlockrc and greenlock.json shall merge as one
+		if (['defaults', 'routes', 'sites'].includes(key)) {
-		// and be called greenlock.json because calling it
-		// .greenlockrc seems to rub people the wrong way
-		if (['manager', 'defaults', 'routes', 'sites'].includes(key)) {
 			return;
 		}
 		config.defaults[key] = config[key];

package-lock.json

@@ -1,6 +1,6 @@
 {
 	"name": "greenlock-manager-fs",
-	"version": "3.1.1",
+	"version": "0.7.0",
 	"lockfileVersion": 1,
 	"requires": true,
 	"dependencies": {
@@ -9,32 +9,6 @@
 			"resolved": "https://registry.npmjs.org/@root/mkdirp/-/mkdirp-1.0.0.tgz",
 			"integrity": "sha512-hxGAYUx5029VggfG+U9naAhQkoMSXtOeXtbql97m3Hi6/sQSRL/4khKZPyOF6w11glyCOU38WCNLu9nUcSjOfA=="
 		},
-		"@root/request": {
-			"version": "1.4.2",
-			"resolved": "https://registry.npmjs.org/@root/request/-/request-1.4.2.tgz",
-			"integrity": "sha512-J8FM4+SJuc7WRC+Jz17m+VT2lgI7HtatHhxN1F2ck5aIKUAxJEaR4u/gLBsgT60mVHevKCjKN0O8115UtJjwLw==",
-			"dev": true
-		},
-		"greenlock-manager-fs": {
-			"version": "3.0.5",
-			"resolved": "https://registry.npmjs.org/greenlock-manager-fs/-/greenlock-manager-fs-3.0.5.tgz",
-			"integrity": "sha512-r/q+tEFuDwklfzPfiGhcIrHuJxMrppC+EseESpu5f0DMokh+1iZVm9nGC/VE7/7GETdOYfEYhhQkmspsi8Gr/A==",
-			"dev": true,
-			"requires": {
-				"@root/mkdirp": "^1.0.0",
-				"safe-replace": "^1.1.0"
-			}
-		},
-		"greenlock-manager-test": {
-			"version": "3.1.1",
-			"resolved": "https://registry.npmjs.org/greenlock-manager-test/-/greenlock-manager-test-3.1.1.tgz",
-			"integrity": "sha512-wZ+Oxn5qTEoN+VDd3Y+kBYZ8MlaLlhm40KwIwfyR90bj08IZpfzE7zGY8SwBEbIx0wNSo6ztDku4Y0gVgxxwCA==",
-			"dev": true,
-			"requires": {
-				"@root/request": "^1.4.1",
-				"greenlock-manager-fs": "^3.0.0"
-			}
-		},
 		"safe-replace": {
 			"version": "1.1.0",
 			"resolved": "https://registry.npmjs.org/safe-replace/-/safe-replace-1.1.0.tgz",

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "greenlock-manager-fs",
-	"version": "3.1.1",
+	"version": "3.0.1",
 	"description": "A simple file-based management strategy for Greenlock",
 	"main": "manager.js",
 	"scripts": {
@@ -27,6 +27,6 @@
 		"safe-replace": "^1.1.0"
 	},
 	"devDependencies": {
-		"greenlock-manager-test": "^3.1.1"
+		"greenlock-manager-test": "^3.0.0"
 	}
 }

tests/index.js

@@ -1,25 +1,19 @@
-"use strict";
+'use strict';
 
-var Tester = require("greenlock-manager-test");
+var Tester = require('greenlock-manager-test');
 
-var Manager = require("../manager.js");
+var Manager = require('../');
 var config = {
-    configFile: "greenlock-manager-test.delete-me.json"
+	configFile: 'greenlock-manager-test.delete-me.json'
 };
 
 Tester.test(Manager, config)
-    .then(function(features) {
+	.then(function() {
-        console.info("PASS");
+		console.log('PASS: Known-good test module passes');
-        console.info();
+	})
-        console.info("Optional Feature Support:");
+	.catch(function(err) {
-        features.forEach(function(feature) {
+		console.error('Oops, you broke it. Here are the details:');
-            console.info(feature.supported ? "✓ (YES)" : "✘ (NO) ", feature.description);
+		console.error(err.stack);
-        });
+		console.error();
-        console.info();
+		console.error("That's all I know.");
-    })
+	});
-    .catch(function(err) {
-        console.error("Oops, you broke it. Here are the details:");
-        console.error(err.stack);
-        console.error();
-        console.error("That's all I know.");
-    });