From d7a4e94306d4300f2edb8ae9bd2105458ce298da Mon Sep 17 00:00:00 2001
From: Vladimir Grichina <vgrichina@gmail.com>
Date: Mon, 11 May 2020 16:30:13 -0700
Subject: [PATCH 1/2] Revert "address windows, mac, linux behavior w/ callback
 and auto-open Wallet"

---
 index.js | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

diff --git a/index.js b/index.js
index 08d89113..8acb48af 100644
--- a/index.js
+++ b/index.js
@@ -77,20 +77,12 @@ exports.login = async function (options) {
 
         // attempt to capture accountId automatically via browser callback
         let tempUrl;
-        // See: https://github.com/near/near-shell/issues/358
-        const isMac = process.platform === 'darwin';
+        const isWin = process.platform === 'win32';
 
         // find a callback URL on the local machine
         try {
-            // capture callback isn't working reliably outside of Mac. This is a workaround to not use it
-            if (isMac) {
+            if (!isWin) { // capture callback is currently not working on windows. This is a workaround to not use it
                 tempUrl = await capture.callback(5000);
-                // if we found a suitable URL, attempt to use it
-                newUrl.searchParams.set('success_url', `http://${tempUrl.hostname}:${tempUrl.port}`);
-                await openUrl(newUrl);
-            } else {
-                // redirect automatically, but do not use the browser callback
-                await openUrl(newUrl);
             }
         } catch (error) {
             // console.error("Failed to find suitable port.", error.message)
@@ -98,6 +90,21 @@ exports.login = async function (options) {
             // silent error is better here
         }
 
+        // if we found a suitable URL, attempt to use it
+        if (tempUrl) {
+            if (process.env.GITPOD_WORKSPACE_URL) {
+                const workspaceUrl = new URL(process.env.GITPOD_WORKSPACE_URL);
+                newUrl.searchParams.set('success_url', `https://${tempUrl.port}-${workspaceUrl.hostname}`);
+                // Browser not opened, as will open automatically for opened port
+            } else {
+                newUrl.searchParams.set('success_url', `http://${tempUrl.hostname}:${tempUrl.port}`);
+                openUrl(newUrl); 
+            }
+        } else if (isWin) {
+            // redirect automatically on windows, but do not use the browser callback
+            openUrl(newUrl);
+        }
+
         console.log(chalk`\n{dim If your browser doesn't automatically open, please visit this URL\n${newUrl.toString()}}`);
 
         const getAccountFromWebpage = async () => {

From 90ffdcb3276388c03c86b2c2564f75b23e702fc5 Mon Sep 17 00:00:00 2001
From: Vladimir Grichina <vgrichina@gmail.com>
Date: Mon, 11 May 2020 16:27:32 -0700
Subject: [PATCH 2/2] Use temporary instead of permanent redirect during login

When using 301 redirect it gets cached in Chrome and next time URL is opened server isn't even hit.
---
 utils/capture-login-success.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/utils/capture-login-success.js b/utils/capture-login-success.js
index 956ed462..aa19e675 100644
--- a/utils/capture-login-success.js
+++ b/utils/capture-login-success.js
@@ -33,7 +33,7 @@ const payload = (fields, { port, hostname }, redirectUrl) => new Promise((resolv
                     resolve(results);
                 });
             } else {
-                res.writeHead(301, { Location: redirectUrl });
+                res.writeHead(302, { Location: redirectUrl });
                 res.end();
             }
         } catch (e) {