Deleting cache-control header on redirect responses, removing `grun…

…t-nsp` module due to audit warnings from `npm`
avoidwork · Jun 4, 2018 · e97e7dc · e97e7dc
1 parent a90bd50
commit e97e7dc
Show file tree

Hide file tree

Showing 4 changed files with 148 additions and 576 deletions.
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -17,9 +17,6 @@ module.exports = function (grunt) {
 				src: ["test/*_test.js"/*, "test/*_test2.js"*/]
 			}
 		},
-		nsp: {
-			package: grunt.file.readJSON("package.json")
-		},
 		watch: {
 			js: {
 				files: ["*.js", "lib/*.js"],
@@ -31,7 +28,6 @@ module.exports = function (grunt) {
 	// tasks
 	grunt.loadNpmTasks("grunt-eslint");
 	grunt.loadNpmTasks("grunt-mocha-test");
-	grunt.loadNpmTasks("grunt-nsp");
 	grunt.loadNpmTasks("grunt-contrib-watch");
 
 	// aliases

diff --git a/lib/tenso.js b/lib/tenso.js
@@ -407,6 +407,11 @@ class Tenso extends HTTPMethods {
 					}
 				});
 
+				if (status >= 300 && status < 400 && rheaders["cache-control"] !== void 0) {
+					res.removeHeader("cache-control");
+					delete headers["cache-control"];
+				}
+
 				if (status !== 416 && req.headers.range !== void 0 && !res.hasHeader("content-range") && headers["content-range"] === void 0) {
 					this.error(req, res, new Error(416), headers).then(resolve, reject);
 				} else {