Do not expose all configuration params to index.html JSON

SC5 · Apr 20, 2015 · b4e011d · b4e011d
1 parent cca9cc6
commit b4e011d
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/styleguide.js b/lib/styleguide.js
@@ -94,6 +94,15 @@ function copyUsedOptionsToJsonConfig(opt, json) {
   return json;
 }
 
+function copyUsedOptionsToInlineJsonConfig(opt, json) {
+  var used = ['title', 'disableEncapsulation', 'disableHtml5Mode'];
+  json.config = {};
+  used.forEach(function(prop) {
+    json.config[prop] = _.cloneDeep(opt[prop]);
+  });
+  return json;
+}
+
 function addFileHashesAndReplaceAbsolutePaths(json) {
   if (!json) {
     return;
@@ -258,7 +267,7 @@ module.exports.generate = function(options) {
             .pipe(mustache({
               title: opt.title,
               extraHead: opt.extraHead,
-              styleguideConfig: JSON.stringify(copyUsedOptionsToJsonConfig(opt, {}).config),
+              styleguideConfig: JSON.stringify(copyUsedOptionsToInlineJsonConfig(opt, {}).config),
               appRoot: opt.appRoot,
               socketIo: opt.server,
               filesConfig: JSON.stringify(opt.filesConfig)

diff --git a/test/integration/structure.test.js b/test/integration/structure.test.js
@@ -64,6 +64,10 @@ describe('index.html', function() {
   it('should contain serialized styleguide configuration', function() {
     expect(indexHtml.contents.toString()).to.contain('var _styleguideConfig = {"appRoot":"/my-styleguide-book"');
   });
+
+  it('should not add extraHead to serialized configuration', function() {
+    expect(indexHtml.contents.toString()).not.to.contain('"extreHead":');
+  });
 });
 
 describe('styleguide_pseudo_styles.css', function() {