Merge pull request #212 from Financial-Times/feature/ignore-sourcemaps

Financial-Times · Mar 8, 2023 · c070599 · c070599
2 parents b7d668d + 48ebcc4
commit c070599
Show file tree

Hide file tree

Showing 5 changed files with 37 additions and 6 deletions.
diff --git a/secret-squirrel.cjs b/secret-squirrel.cjs
@@ -0,0 +1,12 @@
+module.exports = {
+	files: {
+		allow: [
+			'test/fixtures/config/sourcemap.js.map'
+		],
+		allowOverrides: []
+	},
+	strings: {
+		deny: [],
+		denyOverrides: []
+	}
+};
diff --git a/secrets.js b/secrets.js
diff --git a/src/startup.js b/src/startup.js
@@ -45,6 +45,10 @@ function startup(configPath, additionalChecks){
 			return;
 		}
 
+		if(configFile.endsWith('.map')){
+			return;
+		}
+
 		const name = configFile.replace('.js', '');
 		const config = require(path.resolve(configDir, configFile));
 		const healthchecks = new HealthChecks(config, checks);

diff --git a/test/fixtures/config/sourcemap.js.map b/test/fixtures/config/sourcemap.js.map
diff --git a/test/startup.spec.js b/test/startup.spec.js
@@ -10,6 +10,7 @@ describe('Startup', function(){
 
 	it('Should read in the config dir and create new healthcheck objects', function(){
 		const result = startup(path.resolve(__dirname, 'fixtures/config/'));
+		expect(result.get('sourcemap')).not.to.exist;
 		expect(result.get('paywall')).to.exist;
 		expect(result.get('paywall')).to.be.an.instanceOf(HealthChecks);
 	});