From 6db660d992dd547ba6f9e2f33247065b11767c9c Mon Sep 17 00:00:00 2001
From: Ian Bicking <ianb@colorstudy.com>
Date: Tue, 1 Nov 2016 16:00:12 -0500
Subject: [PATCH] Fix #1802, combine parent-helper.js and
 set-content-hosting-origin.js Make the scripts inclusion dependent on there
 being a full page/iframe

---
 server/src/pages/shot/view.js | 3 +--
 server/src/server.js          | 7 +++++--
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/server/src/pages/shot/view.js b/server/src/pages/shot/view.js
index f2c2beedf4..461f82d27c 100644
--- a/server/src/pages/shot/view.js
+++ b/server/src/pages/shot/view.js
@@ -103,8 +103,7 @@ class Head extends React.Component {
         <link rel="alternate" type="application/json+oembed" href={this.props.shot.oembedUrl} title={`${this.props.shot.title} oEmbed`} />
         <meta name="viewport" content="width=device-width, initial-scale=1" />
         {this.socialMetadata()}
-        <script src={ this.props.staticLink("/set-content-hosting-origin.js") } />
-        <script src={ this.props.staticLink("/static/js/parent-helper.js") } />
+        {this.props.shot.showPage ? <script src={ this.props.staticLink("/parent-helper.js") } /> : null}
       </reactruntime.HeadTemplate>);
   }
 
diff --git a/server/src/server.js b/server/src/server.js
index f29851da70..60dfa46e96 100644
--- a/server/src/server.js
+++ b/server/src/server.js
@@ -43,6 +43,7 @@ console.warn = logFactory("warn");
 console.error = logFactory("error");
 
 const path = require('path');
+const { readFileSync } = require('fs');
 const Cookies = require("cookies");
 
 const { Shot } = require("./servershot");
@@ -242,10 +243,12 @@ function sendGaActivation(req, res, hashPage) {
   });
 }
 
-app.get("/set-content-hosting-origin.js", function (req, res) {
+const parentHelperJs = readFileSync(path.join(__dirname, "/static/js/parent-helper.js"), {encoding: "UTF-8"});
+
+app.get("/parent-helper.js", function (req, res) {
   setCache(res);
   let postMessageOrigin = `${req.protocol}://${req.config.contentOrigin}`;
-  let script = `var CONTENT_HOSTING_ORIGIN = "${postMessageOrigin}";`
+  let script = `${parentHelperJs}\nvar CONTENT_HOSTING_ORIGIN = "${postMessageOrigin}";`
   jsResponse(res, script);
 });