[ARVADOS-WORKBENCH2] updated: 1.3.1-486-gfdbec885

Git user git at public.curoverse.com
Wed May 22 19:38:25 UTC 2019


Summary of changes:
 src/views/workbench/fed-login.tsx | 14 +++++++++++---
 1 file changed, 11 insertions(+), 3 deletions(-)

       via  fdbec885a5e37bd22c5e839bc4c6c0f6c7f3b76a (commit)
      from  98adeb294aa8bf1c1691f70ab095d20a96c0d375 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit fdbec885a5e37bd22c5e839bc4c6c0f6c7f3b76a
Author: Peter Amstutz <pamstutz at veritasgenetics.com>
Date:   Wed May 22 15:33:29 2019 -0400

    15255: Log missing workbench2Url value to console.
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <pamstutz at veritasgenetics.com>

diff --git a/src/views/workbench/fed-login.tsx b/src/views/workbench/fed-login.tsx
index d77ddfb0..c64778a3 100644
--- a/src/views/workbench/fed-login.tsx
+++ b/src/views/workbench/fed-login.tsx
@@ -34,13 +34,21 @@ export const FedLogin = connect(mapStateToProps)(
             const [, tokenUuid, token] = apiToken.split("/");
             return <div id={"fedtoken-iframe-div"}>
                 {Object.keys(remoteHostsConfig)
-                    .map((k) => k !== localCluster && remoteHostsConfig[k].workbench2Url &&
-                        <iframe key={k} src={`${remoteHostsConfig[k].workbench2Url}/fedtoken?api_token=${getSaltedToken(k, tokenUuid, token)}`} style={{
+                    .map((k) => {
+                        if (k === localCluster) {
+                            return;
+                        }
+                        if (!remoteHostsConfig[k].workbench2Url) {
+                            console.log(`Cluster ${k} does not define workbench2Url.  Federated login / cross-site linking to ${k} is unavailable.  Tell the admin of ${k} to set Services->Workbench2->ExternalURL in config.yml.`);
+                            return;
+                        }
+                        return <iframe key={k} src={`${remoteHostsConfig[k].workbench2Url}/fedtoken?api_token=${getSaltedToken(k, tokenUuid, token)}`} style={{
                             height: 0,
                             width: 0,
                             visibility: "hidden"
                         }}
-                        />)}
+                        />;
+                    })}
             </div>;
         }
     });

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list