OAUTH: Only start one webbrowser tab for concurrent authentications

Starting a browser tab per authentication attempt makes a mess, so long as
there is something in auth_redirs then the active browser tab will
eventually get redirected to the new auth being added.

Signed-off-by: Jason Gunthorpe <jgg@mellanox.com>

1 files changed, 14 insertions, 0 deletions

diff --git a/cloud_mdir_sync/oauth.py b/cloud_mdir_sync/oauth.py
index 0ab30ae..716e7c0 100644
--- a/cloud_mdir_sync/oauth.py
+++ b/cloud_mdir_sync/oauth.py
@@ -4,6 +4,7 @@ import base64
 import hashlib
 import os
 import secrets
+import webbrowser
 from abc import abstractmethod
 from typing import TYPE_CHECKING, Dict, List, Optional
 
@@ -59,6 +60,19 @@ class WebServer(object):
         redirects back to the localhost server.  The final query paremeters
         will be returned by this function"""
         queue = asyncio.Queue()
+
+        # If this is the first auth to start then automatically launch a
+        # browser, otherwise assume the already running browser will take care
+        # of things
+        if not self.auth_redirs:
+            print(
+                f"Goto {self.url} in a web browser to authenticate (opening browser)"
+            )
+            webbrowser.open(url)
+        else:
+            print(
+                f"Goto {self.url} in a web browser to authenticate (reusing browser)"
+            )
         self.auth_redirs[state] = (url, queue, redir_url)
         return await queue.get()