Merge pull request #5 from BrianEmilius/sanitize-paths

Sanitize paths

CHANGELOG.md

@@ -8,6 +8,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Sanitize paths to prevent saving logs in dangerous places.
 * Support for deeper log paths (currently, the log path can only be one directory deep).
 
+## [2.0.0] - 2018-11-27
+### Changed
+* `logErrors` parameter list has been changed. You can now only define a name for the log file. See documentation.
+
+### Removed
+* Options parameter: You can no longer define options for logErrors in an object literal. Instead, use `logname`. See documentation.
+
 ## [1.1.1] - 2018-11-27
 ### Added
 * Secret easter-egg

lib/log-errors.js

@@ -17,20 +17,17 @@ exports = module.exports = logErrors;
 
 /**
  *
- * @param {Object} options
+ * @param {String} logName
  * @return {Function}
  * @api public
  */
 
-function logErrors(options = {
-	'path': './logs',
-	'logName': 'errors.log'
-}) {
-	if (!fs.existsSync(options.path))
-		fs.mkdirSync(options.path);
+function logErrors(logName = 'express-errors.log') {
+	if (!fs.existsSync('./logs'))
+		fs.mkdirSync('./logs');
 	return function (err, req, res, next) {
 		if (err) {
-			const fullPath = path.join(options.path, options.logName);
+			const fullPath = path.join('.', 'logs', logName);
 			if (!fs.existsSync(fullPath))
 				fs.writeFileSync(fullPath, '');
 			const oldLog = fs.readFileSync(fullPath);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "express-log-errors",
-  "version": "1.1.1",
+  "version": "2.0.0",
   "description": "express middleware for logging errors from routes",
   "keywords": [
     "expressjs",