browser: more failsafe cookie persistence

Author: László Vaskó

CHANGELOG.md

@@ -2,7 +2,12 @@
 
 ## vNext
 
+- More failsafe method to reuse existing authentication sessions, so that
+  entering password/MFA token may not be needed at all. Persisting HTTP cookies
+  were prone to a race condition which is hopefully mitigated by this change.
+
 ## v0.7.2
+
 - Update dependencies `keyring` and `importlib-metadata`
   ([pull/52](https://github.com/vlaci/openconnect-sso/pull/52))
 

openconnect_sso/browser/webengine_process.py

@@ -129,7 +129,11 @@ def on_sigterm(signum, frame):
     # See: https://github.com/qutebrowser/qutebrowser/commit/8d55d093f29008b268569cdec28b700a8c42d761
     cookie = QNetworkCookie()
     QWebEngineProfile.defaultProfile().cookieStore().deleteCookie(cookie)
-    QApplication.quit()
+
+    # Give some time to actually save cookies
+    exit_timer = QTimer(app)
+    exit_timer.timeout.connect(QApplication.quit)
+    exit_timer.start(1000)  # ms
 
 
 class WebBrowser(QWebEngineView):