From 0fce0e3b31592736a3ee5898cda9456e5bcb42c1 Mon Sep 17 00:00:00 2001
From: Jiri Zajic <jiri.zajic@gooddata.com>
Date: Mon, 18 Oct 2021 17:10:39 +0200
Subject: [PATCH] geochart - Proxy - no authentication needed

---
 geochart/README.md                       |  4 +--
 geochart/src/components/Header/Header.js |  4 +--
 geochart/src/constants.js                |  2 +-
 geochart/src/contexts/Auth/backend.js    | 17 ++++-----
 geochart/src/routes/AppRouter.js         | 18 +++++-----
 geochart/src/routes/Home.js              |  2 +-
 geochart/src/setupProxy.js               | 44 ------------------------
 7 files changed, 20 insertions(+), 71 deletions(-)
 delete mode 100644 geochart/src/setupProxy.js

diff --git a/geochart/README.md b/geochart/README.md
index edd7644d..6e05d4e7 100644
--- a/geochart/README.md
+++ b/geochart/README.md
@@ -4,9 +4,7 @@ This demonstrates how you can feed data from the [GoodData](https://www.gooddata
 
 ## Backend
 
-This app is designed to connect to domain https://developer.na.gooddata.com/, workspace `xms7ga4tf3g3nzucd8380o2bev8oeknp`.
-
-For access, you can register at https://gooddata-examples.herokuapp.com/.
+This app is designed to connect to workspace `xms7ga4tf3g3nzucd8380o2bev8oeknp` on https://live-examples-proxy.herokuapp.com/ domain, and does not require authentication.
 
 ## Live demo
 
diff --git a/geochart/src/components/Header/Header.js b/geochart/src/components/Header/Header.js
index 0cbe60d0..e93cefe7 100644
--- a/geochart/src/components/Header/Header.js
+++ b/geochart/src/components/Header/Header.js
@@ -8,7 +8,7 @@ import { AuthStatus } from "../../contexts/Auth/state";
 
 import Logo from "./Logo";
 import Links from "./Links";
-import Aside from "./Aside";
+// import Aside from "./Aside";
 
 import styles from "./Header.module.scss";
 
@@ -91,7 +91,7 @@ const Header = () => {
                 </>
             )}
 
-            <Aside />
+            {/* <Aside /> */}
         </nav>
     );
 };
diff --git a/geochart/src/constants.js b/geochart/src/constants.js
index eb571a17..cf6e24c7 100644
--- a/geochart/src/constants.js
+++ b/geochart/src/constants.js
@@ -3,7 +3,7 @@ export const appName = "geochart";
 
 // The URL of analytical GoodData backend with workspaces, insights, metrics, attributes, datasets, and users
 // that will be available to the application.
-export const backend = "https://developer.na.gooddata.com";
+export const backend = "https://live-examples-proxy.herokuapp.com/";
 
 // The ID of workspace that is selected by default in the optional workspace picker component (located in the web
 // application header). It is also a default setting for script that refreshes MD data used by the application.
diff --git a/geochart/src/contexts/Auth/backend.js b/geochart/src/contexts/Auth/backend.js
index ddccc866..e2cf588b 100644
--- a/geochart/src/contexts/Auth/backend.js
+++ b/geochart/src/contexts/Auth/backend.js
@@ -1,16 +1,11 @@
 import bearFactory, {
     FixedLoginAndPasswordAuthProvider,
-    ContextDeferredAuthProvider,
+    AnonymousAuthProvider,
 } from "@gooddata/sdk-backend-bear";
+import { backend } from "../../constants";
 
-import { backend as hostname } from "../../constants";
+export const createBackend = () =>
+    bearFactory({ hostname: backend }).withAuthentication(new AnonymousAuthProvider());
 
-const backendConfig = process.env.REACT_APP_SET_HOSTNAME ? { hostname } : {};
-
-export const createBackend = () => {
-    return bearFactory(backendConfig).withAuthentication(new ContextDeferredAuthProvider());
-};
-
-export const backendWithCredentials = (backend, username, password) => {
-    return backend.withAuthentication(new FixedLoginAndPasswordAuthProvider(username, password));
-};
+export const backendWithCredentials = (backend, username, password) =>
+    backend.withAuthentication(new FixedLoginAndPasswordAuthProvider(username, password));
diff --git a/geochart/src/routes/AppRouter.js b/geochart/src/routes/AppRouter.js
index 6ee0e27b..6374d9cb 100644
--- a/geochart/src/routes/AppRouter.js
+++ b/geochart/src/routes/AppRouter.js
@@ -1,5 +1,5 @@
 import React from "react";
-import { HashRouter as Router, Route, Redirect } from "react-router-dom";
+import { HashRouter as Router, Route } from "react-router-dom";
 
 import { WorkspaceProvider } from "../contexts/Workspace";
 import Page from "../components/Page";
@@ -12,13 +12,13 @@ import Home from "./Home";
 import styles from "./AppRouter.module.scss";
 
 // Uncomment these lines if you want to redirect unauthorized users to login form
-import { useAuth } from "../contexts/Auth";
-import { AuthStatus } from "../contexts/Auth/state";
-const RedirectIfNotLoggedIn = () => {
-    const auth = useAuth();
-    const shouldRedirectToLogin = auth.authStatus === AuthStatus.UNAUTHORIZED;
-    return shouldRedirectToLogin ? <Route component={() => <Redirect to="/login" />} /> : null;
-};
+// import { useAuth } from "../contexts/Auth";
+// import { AuthStatus } from "../contexts/Auth/state";
+// const RedirectIfNotLoggedIn = () => {
+//     const auth = useAuth();
+//     const shouldRedirectToLogin = auth.authStatus === AuthStatus.UNAUTHORIZED;
+//     return shouldRedirectToLogin ? <Route component={() => <Redirect to="/login" />} /> : null;
+// };
 
 const AppRouter = () => {
     return (
@@ -31,7 +31,7 @@ const AppRouter = () => {
                     <Route exact path="/dashboard" component={() => <Page>Dashboard</Page>} />
                     <Route exact path="/login" component={Login} />
                     <Route exact path="/logout" component={Logout} />
-                    <RedirectIfNotLoggedIn />
+                    {/* <RedirectIfNotLoggedIn /> */}
                 </WorkspaceProvider>
             </Router>
         </div>
diff --git a/geochart/src/routes/Home.js b/geochart/src/routes/Home.js
index e4dc45fa..8aef9c49 100644
--- a/geochart/src/routes/Home.js
+++ b/geochart/src/routes/Home.js
@@ -15,7 +15,7 @@ const Home = () => {
                     }
 
                     if (error) {
-                        return <p>Execution failed; {error}</p>;
+                        return <p>Execution failed; {error.toString()}</p>;
                     }
 
                     if (result) {
diff --git a/geochart/src/setupProxy.js b/geochart/src/setupProxy.js
deleted file mode 100644
index 31422387..00000000
--- a/geochart/src/setupProxy.js
+++ /dev/null
@@ -1,44 +0,0 @@
-const proxy = require("http-proxy-middleware");
-
-require("@babel/register")({
-    presets: ["@babel/preset-typescript", "@babel/preset-env"],
-    plugins: ["add-module-exports"],
-    extensions: [".ts", ".js"],
-});
-
-const constants = require("./constants");
-
-const domain = constants.backend;
-module.exports = function(app) {
-    app.use(
-        proxy("/gdc", {
-            changeOrigin: true,
-            cookieDomainRewrite: "localhost",
-            secure: false,
-            target: domain,
-            headers: {
-                host: domain.replace(/https:\/\//, ""),
-                origin: null,
-                // This is essential for Tiger backends. To ensure 401 flies when not authenticated and using proxy
-                "X-Requested-With": "XMLHttpRequest",
-            },
-            onProxyReq: function(proxyReq, _req, _res) {
-                proxyReq.setHeader("accept-encoding", "identity");
-            },
-        }),
-    );
-    app.use(
-        proxy("/*.html", {
-            changeOrigin: true,
-            secure: false,
-            target: domain,
-        }),
-    );
-    app.use(
-        proxy("/packages/*.{js,css}", {
-            changeOrigin: true,
-            secure: false,
-            target: domain,
-        }),
-    );
-};