From bde60c4368182699ff0d841fb8640e91dfbb3329 Mon Sep 17 00:00:00 2001
From: Christopher Johnson <chjohnson39@gmail.com>
Date: Tue, 9 Jul 2019 12:54:12 +0200
Subject: [PATCH 1/3] removes react-iframe-comm dependency / replaces with
 local updated implementation

---
 .../mirador/window_actions.test.js            |   2 +-
 package.json                                  |   1 -
 src/components/AccessTokenSender.js           |   2 +-
 src/components/WindowAuthenticationControl.js |   2 +-
 src/lib/IFrameComm.js                         | 200 ++++++++++++++++++
 src/lib/index.js                              |   1 +
 6 files changed, 204 insertions(+), 4 deletions(-)
 create mode 100644 src/lib/IFrameComm.js
 create mode 100644 src/lib/index.js

diff --git a/__tests__/integration/mirador/window_actions.test.js b/__tests__/integration/mirador/window_actions.test.js
index 0f62f257a2..58df7a6e33 100644
--- a/__tests__/integration/mirador/window_actions.test.js
+++ b/__tests__/integration/mirador/window_actions.test.js
@@ -19,5 +19,5 @@ describe('Window actions', () => {
     )); // only default configed windows found
     await page.waitFor(1000);
     await expect(numWindows).toBe(2);
-  });
+  }, 10000);
 });
diff --git a/package.json b/package.json
index 2784a76738..1c3307cf27 100644
--- a/package.json
+++ b/package.json
@@ -52,7 +52,6 @@
     "react-copy-to-clipboard": "^5.0.1",
     "react-full-screen": "^0.2.4",
     "react-i18next": "^10.11.2",
-    "react-iframe-comm": "^1.2.2",
     "react-image": "^2.1.3",
     "react-mosaic-component": "^3.2.0",
     "react-placeholder": "^3.0.1",
diff --git a/src/components/AccessTokenSender.js b/src/components/AccessTokenSender.js
index dae3bccdb3..06f510d0ea 100644
--- a/src/components/AccessTokenSender.js
+++ b/src/components/AccessTokenSender.js
@@ -1,6 +1,6 @@
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';
-import IframeComm from 'react-iframe-comm';
+import { IframeComm } from '../lib/IFrameComm';
 
 /**
  * Opens a new window for click
diff --git a/src/components/WindowAuthenticationControl.js b/src/components/WindowAuthenticationControl.js
index 87f70ab727..92786bbdc0 100644
--- a/src/components/WindowAuthenticationControl.js
+++ b/src/components/WindowAuthenticationControl.js
@@ -91,7 +91,7 @@ export class WindowAuthenticationControl extends Component {
       <Paper square elevation={4} color="secondary" classes={{ root: classes.paper }}>
         <Button fullWidth className={classes.topBar} onClick={hasCollapsedContent ? this.handleClickOpen : this.handleConfirm} component="div" color="inherit">
           <LockIcon className={classes.icon} />
-          <Typography className={classes.label} component="h3" variant="body1" color="inherit" inline>
+          <Typography className={classes.label} component="h3" variant="body1" color="inherit" inline="true">
             <SanitizedHtml htmlString={(isInFailureState ? failureHeader : label) || t('authenticationRequired')} ruleSet="iiif" />
           </Typography>
           <span className={classes.fauxButton}>
diff --git a/src/lib/IFrameComm.js b/src/lib/IFrameComm.js
new file mode 100644
index 0000000000..eed609b79b
--- /dev/null
+++ b/src/lib/IFrameComm.js
@@ -0,0 +1,200 @@
+import React, { Component } from 'react';
+import PropTypes from 'prop-types';
+
+/**
+ *
+ */
+export class IframeComm extends Component {
+  /**
+   *
+   */
+  constructor() {
+    super();
+    this.onReceiveMessage = this.onReceiveMessage.bind(this);
+    this.onLoad = this.onLoad.bind(this);
+    this.sendMessage = this.sendMessage.bind(this);
+  }
+
+  /**
+   *
+   */
+  componentDidMount() {
+    window.addEventListener('message', this.onReceiveMessage);
+    this.frame.addEventListener('load', this.onLoad);
+  }
+
+  /**
+   *
+   * @param prevProps
+   * @param prevState
+   */
+  componentDidUpdate(prevProps, prevState) {
+    const { postMessageData } = this.props;
+    if (prevProps.postMessageData !== postMessageData) {
+      // send a message if postMessageData changed
+      this.sendMessage(postMessageData);
+    }
+  }
+
+  /**
+   *
+   */
+  componentWillUnmount() {
+    window.removeEventListener('message', this.onReceiveMessage, false);
+  }
+
+  /**
+   *
+   * @param event
+   */
+  onReceiveMessage(event) {
+    const { handleReceiveMessage } = this.props;
+    if (handleReceiveMessage) {
+      handleReceiveMessage(event);
+    }
+  }
+
+  /**
+   *
+   */
+  onLoad() {
+    const { handleReady } = this.props;
+    if (handleReady) {
+      handleReady();
+    }
+    // TODO: Look into doing a syn-ack TCP-like handshake
+    //       to make sure iFrame is ready to REALLY accept messages, not just loaded.
+    // send intial props when iframe loads
+    const { postMessageData } = this.props;
+    this.sendMessage(postMessageData);
+  }
+
+  /**
+   *
+   * @param data
+   * @returns {string|*}
+   */
+  serializePostMessageData(data) {
+    // Rely on the browser's built-in structured clone algorithm for serialization of the
+    // message as described in
+    // https://developer.mozilla.org/en-US/docs/Web/API/Window/postMessage
+    const { serializeMessage } = this.props;
+    if (!serializeMessage) {
+      return data;
+    }
+
+    // To be on the safe side we can also ignore the browser's built-in serialization feature
+    // and serialize the data manually.
+    if (typeof data === 'object') {
+      return JSON.stringify(data);
+    } if (typeof data === 'string') {
+      return data;
+    }
+    return `${data}`;
+  }
+
+  /**
+   *
+   * @param postMessageData
+   */
+  sendMessage(postMessageData) {
+    // Using postMessage data from props will result in a subtle but deadly bug,
+    // where old data from props is being sent instead of new postMessageData.
+    // This is because data sent from componentWillReceiveProps is not yet
+    // in props but only in nextProps.
+    const { targetOrigin } = this.props;
+    const serializedData = this.serializePostMessageData(postMessageData);
+    this.frame.contentWindow.postMessage(serializedData, targetOrigin);
+  }
+
+  /**
+   *
+   * @returns {*}
+   */
+  render() {
+    const { attributes } = this.props;
+    // define some sensible defaults for our iframe attributes
+    const defaultAttributes = {
+      allowFullScreen: false,
+      frameBorder: 0,
+    };
+    // then merge in the user's attributes with our defaults
+    const mergedAttributes = Object.assign(
+      {},
+      defaultAttributes,
+      attributes,
+    );
+    return (
+      <iframe //eslint-disable-line
+        ref={(el) => {
+          this.frame = el;
+        }}
+        {...mergedAttributes}
+      />
+    );
+  }
+}
+
+IframeComm.defaultProps = {
+  attributes: {},
+  handleReady: {},
+  handleReceiveMessage: {},
+  serializeMessage: true,
+  targetOrigin: '*',
+};
+
+IframeComm.propTypes = {
+  /*
+      Iframe Attributes
+      https://developer.mozilla.org/en-US/docs/Web/HTML/Element/iframe#Attributes
+      React Supported Attributes
+      https://facebook.github.io/react/docs/dom-elements.html#all-supported-html-attributes
+      Note: attributes are camelCase, not all lowercase as usually defined.
+  */
+  attributes: PropTypes.shape({
+    allowFullScreen: PropTypes.oneOfType([
+      PropTypes.string,
+      PropTypes.bool,
+    ]),
+    frameBorder: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
+    height: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
+    name: PropTypes.string,
+    sandbox: PropTypes.string,
+    scrolling: PropTypes.string,
+    // https://www.html5rocks.com/en/tutorials/security/sandboxed-iframes/
+    src: PropTypes.string.isRequired,
+    srcDoc: PropTypes.string,
+    width: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
+  }),
+  /*
+      Callback function called when iframe loads. We're simply listening
+      to the iframe's `window.onload`. To ensure communication code in your iframe
+       is totally loaded, you can implement a syn-ack TCP-like handshake using
+      `postMessageData` and `handleReceiveMessage`.
+  */
+  handleReady: PropTypes.func,
+  // Callback function called when iFrame sends the parent window a message.
+  handleReceiveMessage: PropTypes.func,
+
+  /*
+      You can pass it anything you want, we'll serialize to a string
+      preferablly use a simple string message or an object.
+      If you use an object, you need to follow the same naming convention
+      in the iframe so you can parse it accordingly.
+   */
+  postMessageData: PropTypes.any.isRequired, //eslint-disable-line
+
+  /*
+      Enable use of the browser's built-in structured clone algorithm for serialization
+      by settings this to `false`.
+      Default is `true`, using our built in logic for serializing everything to a string.
+  */
+  serializeMessage: PropTypes.bool,
+
+  /*
+      Always provide a specific targetOrigin, not *, if you know where the other window's
+       document should be located. Failing to provide a specific target discloses the data
+        you send to any interested malicious site.
+   */
+  targetOrigin: PropTypes.string,
+};
diff --git a/src/lib/index.js b/src/lib/index.js
new file mode 100644
index 0000000000..ae7ac54259
--- /dev/null
+++ b/src/lib/index.js
@@ -0,0 +1 @@
+export * from './IFrameComm';

From 1e133b51e96b3432edf76cfef152cff1824b993a Mon Sep 17 00:00:00 2001
From: Christopher Johnson <chjohnson39@gmail.com>
Date: Tue, 9 Jul 2019 13:15:03 +0200
Subject: [PATCH 2/3] adds IFrameComm test

---
 __tests__/src/lib/IFrameComm.test.js | 71 ++++++++++++++++++++++++++++
 src/components/AccessTokenSender.js  |  1 +
 src/lib/IFrameComm.js                |  9 ++--
 3 files changed, 77 insertions(+), 4 deletions(-)
 create mode 100644 __tests__/src/lib/IFrameComm.test.js

diff --git a/__tests__/src/lib/IFrameComm.test.js b/__tests__/src/lib/IFrameComm.test.js
new file mode 100644
index 0000000000..d84299b879
--- /dev/null
+++ b/__tests__/src/lib/IFrameComm.test.js
@@ -0,0 +1,71 @@
+import React from 'react';
+import { mount } from 'enzyme';
+import { IframeComm } from '../../../src/lib/IFrameComm';
+
+/**
+ *
+ * @param props
+ * @returns {*}
+ */
+function createWrapper(props) {
+  return mount(
+    <IframeComm
+      handleReady={() => {}}
+      handleReceiveMessage={() => {}}
+      postMessageData="this is a test"
+      {...props}
+    />,
+  );
+}
+
+let receivedMessage = {};
+const mockIFrameContents = {
+  contentWindow: {
+    postMessage: (postMessageData) => {
+      receivedMessage = postMessageData;
+    },
+  },
+};
+
+describe('IFrameComm', () => {
+  let wrapper;
+  it('renders properly', () => {
+    const handleReady = jest.fn();
+    const handleReceiveMessage = jest.fn();
+    wrapper = createWrapper({ handleReady, handleReceiveMessage });
+    expect(wrapper.find(IframeComm).length).toBe(1);
+    wrapper.instance().setIFrameElement(mockIFrameContents);
+    wrapper.instance().onLoad();
+    expect(handleReady).toHaveBeenCalled();
+    expect(receivedMessage === 'this is a test').toBeTruthy();
+    wrapper.instance().onReceiveMessage();
+    expect(handleReceiveMessage).toHaveBeenCalled();
+  });
+  it('updates props', () => {
+    wrapper = createWrapper();
+    wrapper.instance().setIFrameElement(mockIFrameContents);
+    wrapper.instance().onLoad();
+    wrapper.setProps({ postMessageData: 'this is another test' });
+    wrapper.instance().setIFrameElement(mockIFrameContents);
+    wrapper.instance().onLoad();
+    expect(receivedMessage === 'this is another test').toBeTruthy();
+  });
+  it('returns data with no serialization option', () => {
+    wrapper = createWrapper({ serializeMessage: false });
+    wrapper.instance().setIFrameElement(mockIFrameContents);
+    wrapper.instance().onLoad();
+    expect(receivedMessage === 'this is a test').toBeTruthy();
+  });
+  it('returns data with when payload is an object', () => {
+    wrapper = createWrapper({ postMessageData: { object: 'this is an object' } });
+    wrapper.instance().setIFrameElement(mockIFrameContents);
+    wrapper.instance().onLoad();
+    expect(receivedMessage === '{"object":"this is an object"}').toBeTruthy();
+  });
+  it('ummounts', () => {
+    wrapper = createWrapper();
+    const componentWillUnmount = jest.spyOn(wrapper.instance(), 'componentWillUnmount');
+    wrapper.unmount();
+    expect(componentWillUnmount).toHaveBeenCalled();
+  });
+});
diff --git a/src/components/AccessTokenSender.js b/src/components/AccessTokenSender.js
index 06f510d0ea..80e87c613d 100644
--- a/src/components/AccessTokenSender.js
+++ b/src/components/AccessTokenSender.js
@@ -32,6 +32,7 @@ export class AccessTokenSender extends Component {
       <IframeComm
         attributes={{ src: `${url}?origin=${window.origin}&messageId=${url}` }}
         handleReceiveMessage={this.onReceiveAccessTokenMessage}
+        postMessageData=""
       />
     );
   }
diff --git a/src/lib/IFrameComm.js b/src/lib/IFrameComm.js
index eed609b79b..8f0d636940 100644
--- a/src/lib/IFrameComm.js
+++ b/src/lib/IFrameComm.js
@@ -13,6 +13,9 @@ export class IframeComm extends Component {
     this.onReceiveMessage = this.onReceiveMessage.bind(this);
     this.onLoad = this.onLoad.bind(this);
     this.sendMessage = this.sendMessage.bind(this);
+    this.setIFrameElement = (el) => {
+      this.frame = el;
+    };
   }
 
   /**
@@ -64,7 +67,7 @@ export class IframeComm extends Component {
     }
     // TODO: Look into doing a syn-ack TCP-like handshake
     //       to make sure iFrame is ready to REALLY accept messages, not just loaded.
-    // send intial props when iframe loads
+    // send initial props when iframe loads
     const { postMessageData } = this.props;
     this.sendMessage(postMessageData);
   }
@@ -126,9 +129,7 @@ export class IframeComm extends Component {
     );
     return (
       <iframe //eslint-disable-line
-        ref={(el) => {
-          this.frame = el;
-        }}
+        ref={this.setIFrameElement}
         {...mergedAttributes}
       />
     );

From 64410b8087b79911b178c925545fdaf7ccc92dff Mon Sep 17 00:00:00 2001
From: Christopher Johnson <chjohnson39@gmail.com>
Date: Fri, 12 Jul 2019 17:15:21 +0200
Subject: [PATCH 3/3] adds license

---
 src/lib/IFrameComm.js | 24 ++++++++++++++++++++++++
 1 file changed, 24 insertions(+)

diff --git a/src/lib/IFrameComm.js b/src/lib/IFrameComm.js
index 8f0d636940..14dc00680c 100644
--- a/src/lib/IFrameComm.js
+++ b/src/lib/IFrameComm.js
@@ -1,3 +1,27 @@
+/**
+  MIT License
+
+  Copyright (c) 2017 Petar Bojinov
+
+  Permission is hereby granted, free of charge, to any person obtaining a copy
+  of this software and associated documentation files (the "Software"), to deal
+  in the Software without restriction, including without limitation the rights
+  to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+  copies of the Software, and to permit persons to whom the Software is
+  furnished to do so, subject to the following conditions:
+
+    The above copyright notice and this permission notice shall be included in all
+  copies or substantial portions of the Software.
+
+    THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+    FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+  AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+  LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+    OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+  SOFTWARE.
+ */
+
 import React, { Component } from 'react';
 import PropTypes from 'prop-types';