[Libreoffice-commits] online.git: Branch 'distro/collabora/collabora-online-2-1' - wsd/DocumentBroker.cpp

Pranav Kant pranavk at collabora.co.uk
Fri Jul 28 16:15:52 UTC 2017


 wsd/DocumentBroker.cpp |    8 ++++++++
 1 file changed, 8 insertions(+)

New commits:
commit b80f27f552b507e0763dfa1d5645b362ad39e78c
Author: Pranav Kant <pranavk at collabora.co.uk>
Date:   Thu Jun 22 17:04:59 2017 +0530

    Update PostMessageOrigin if SSL termination is on too
    
    Change-Id: I887ab368fec62a5efefde4da3762c47dd0bf66d7
    Reviewed-on: https://gerrit.libreoffice.org/39100
    Reviewed-by: pranavk <pranavk at collabora.co.uk>
    Tested-by: pranavk <pranavk at collabora.co.uk>
    (cherry picked from commit a03f69a16a05aba90de600f0f92e2b89af2bf591)
    Reviewed-on: https://gerrit.libreoffice.org/39262
    Reviewed-by: Jan Holesovsky <kendy at collabora.com>
    Tested-by: Jan Holesovsky <kendy at collabora.com>

diff --git a/wsd/DocumentBroker.cpp b/wsd/DocumentBroker.cpp
index fa82995c..5407caed 100644
--- a/wsd/DocumentBroker.cpp
+++ b/wsd/DocumentBroker.cpp
@@ -417,6 +417,14 @@ bool DocumentBroker::load(const std::shared_ptr<ClientSession>& session, const s
         Object::Ptr wopiInfo = new Object();
         if (!wopifileinfo->_postMessageOrigin.empty())
         {
+            // Update the scheme to https if ssl or ssl termination is on
+            if (wopifileinfo->_postMessageOrigin.substr(0, 7) == "http://" &&
+                (LOOLWSD::isSSLEnabled() || LOOLWSD::isSSLTermination()))
+            {
+                wopifileinfo->_postMessageOrigin.replace(0, 4, "https");
+                LOG_DBG("Updating PostMessageOrgin scheme to HTTPS. Updated origin is [" << wopifileinfo->_postMessageOrigin << "].");
+            }
+
             wopiInfo->set("PostMessageOrigin", wopifileinfo->_postMessageOrigin);
         }
 


More information about the Libreoffice-commits mailing list