0.10.1

Release 0.10.1

Author: xorialexandrov

docs/changelog.md

@@ -2,6 +2,10 @@
 
 Versions follow [Semantic Versioning](https://semver.org/): `<major>.<minor>.<patch>`.
 
+## HardPy 0.10.1
+
+* Fix **StandCloud** authorization process in Windows.
+
 ## HardPy 0.10.0
 
 * Add the `[stand_cloud]` section to the **hardpy.toml** configuration file.

hardpy/common/stand_cloud/registration.py

@@ -12,6 +12,8 @@
 import socket
 import subprocess
 import sys
+from contextlib import suppress
+from platform import system
 from time import sleep
 from typing import TYPE_CHECKING
 from urllib.parse import urlencode
@@ -109,6 +111,12 @@ def logout() -> bool:
             delete_password(service_name, cred.username)
     except KeyringError:
         return False
+    # TODO(xorialexandrov): fix keyring clearing
+    # Windows does not clear refresh token by itself
+    if system() == "Windows":
+        storage_keyring, _ = get_token_store()
+        with suppress(KeyringError):
+            storage_keyring.delete_password(service_name, "refresh_token")
     return True
 
 
@@ -147,6 +155,18 @@ def _create_callback_process(port: str) -> subprocess.Popen:
         "--log-level=error",
     ]
 
+    if system() == "Windows":
+        env = os.environ.copy()
+        env["PYTHONUNBUFFERED"] = "1"
+
+        return subprocess.Popen(  # noqa: S603
+            args,
+            stdout=subprocess.PIPE,
+            bufsize=1,
+            universal_newlines=True,
+            env=env,
+            creationflags=subprocess.CREATE_NEW_PROCESS_GROUP,  # type: ignore
+        )
     return subprocess.Popen(  # noqa: S603
         args,
         stdout=subprocess.PIPE,
@@ -187,7 +207,14 @@ def _check_incorrect_response(response: dict, state: str) -> None:
         print(f"{error}: {error_description}")
         sys.exit(1)
 
-def _par_data(code_verifier: str, client_id: str, port: str, state: str, api_url: str) -> dict:
+
+def _par_data(
+    code_verifier: str,
+    client_id: str,
+    port: str,
+    state: str,
+    api_url: str,
+) -> dict:
     """Create pushed authorization request data.
 
     Returns:

hardpy/common/stand_cloud/token_storage.py

@@ -2,6 +2,7 @@
 # GNU General Public License v3.0 (see LICENSE or https://www.gnu.org/licenses/gpl-3.0.txt)
 from __future__ import annotations
 
+from platform import system
 from typing import TYPE_CHECKING
 
 from keyring.core import load_keyring
@@ -16,7 +17,10 @@ def get_token_store() -> tuple[KeyringBackend, KeyringBackend]:
     Returns:
         tuple[KeyringBackend, KeyringBackend]: token store
     """
-    storage_keyring = load_keyring("keyring.backends.SecretService.Keyring")
+    if system() == "Linux":
+        storage_keyring = load_keyring("keyring.backends.SecretService.Keyring")
+    elif system() == "Windows":
+        storage_keyring = load_keyring("keyring.backends.Windows.WinVaultKeyring")
     # TODO(xorialexandrov): add memory keyring or other store
     mem_keyring = storage_keyring
 

pyproject.toml

@@ -3,7 +3,7 @@
 
 [project]
     name = "hardpy"
-    version = "0.10.0"
+    version = "0.10.1"
     description = "HardPy library for device testing"
     license = "GPL-3.0-or-later"
     authors = [{ name = "Everypin", email = "info@everypin.io" }]