fix: resolve Linux keyring issues and improve User-Agent compatibility

Author: setyanoegraha

hmv/modules/auth.py

@@ -3,43 +3,70 @@
 import os
 import json
 from rich.console import Console
+from keyring.errors import NoKeyringError
 
 console = Console()
 
 class AuthManager:
     def __init__(self):
-        self.app_name = "hmv-cli-pro"
+        self.app_name = "hmv-cli"
         self.config_dir = os.path.expanduser("~/.hmv")
         self.config_file = os.path.join(self.config_dir, "config.json")
-
+        
         if not os.path.exists(self.config_dir):
             os.makedirs(self.config_dir)
 
     def save_credentials(self, username, password):
         """Save username to file and password to system keychain."""
-        with open(self.config_file, "w") as f:
-            json.dump({"username": username}, f)
+        try:
+            with open(self.config_file, "w") as f:
+                json.dump({"username": username}, f)
 
-        keyring.set_password(self.app_name, username, password)
-        console.print("[bold green][+][/bold green] Credentials saved successfully to system vault!")
+            keyring.set_password(self.app_name, username, password)
+            console.print("[bold green][+][/bold green] Credentials saved successfully!")
+            
+        except NoKeyringError:
+            console.print("[bold red][!][/bold red] Error: No recommended keyring backend was found.")
+            console.print("[yellow][*][/yellow] Linux users: Please install [bold white]keyrings.alt[/bold white] or a secret service.")
+            console.print("    Command: [cyan]pip install keyrings.alt[/cyan]")
+        except Exception as e:
+            console.print(f"[bold red][!][/bold red] Failed to save password to system vault.")
+            console.print(f"[dim]Error detail: {e}[/dim]")
 
     async def get_session(self):
         """Establish and return an authenticated HTTPX session."""
         if not os.path.exists(self.config_file):
             console.print("[bold red][!][/bold red] Configuration missing. Please run 'hmv config' first.")
             return None
 
-        with open(self.config_file, "r") as f:
-            username = json.load(f).get("username")
+        try:
+            with open(self.config_file, "r") as f:
+                username = json.load(f).get("username")
 
-        password = keyring.get_password(self.app_name, username)
+            password = keyring.get_password(self.app_name, username)
+        except NoKeyringError:
+            console.print("[bold red][!][/bold red] Keyring backend not found. Run [cyan]pip install keyrings.alt[/cyan]")
+            return None
+        except Exception as e:
+            console.print(f"[bold red][!][/bold red] Error accessing vault: {e}")
+            return None
+
+        if not password:
+            console.print("[bold red][!][/bold red] Password not found. Please run 'hmv config' again.")
+            return None
+
+        user_agent = (
+            "Mozilla/5.0 (Windows NT 10.0; Win64; x64) "
+            "AppleWebKit/537.36 (KHTML, like Gecko) "
+            "Chrome/120.0.0.0 Safari/537.36 HMV-CLI/0.1.0"
+        )
 
         timeout = httpx.Timeout(60.0, connect=15.0)
         client = httpx.AsyncClient(
             base_url="https://hackmyvm.eu", 
             follow_redirects=True, 
             timeout=timeout,
-            headers={"User-Agent": "HMV-CLI-Pro/0.1.0"}
+            headers={"User-Agent": user_agent}
         )
 
         try:
@@ -54,11 +81,7 @@ async def get_session(self):
             console.print("[bold red][!][/bold red] Authentication failed. Check your credentials.")
             await client.aclose()
             return None
-        except httpx.TimeoutException:
-            console.print("[bold red][!][/bold red] Authentication timed out. The server is being slow.")
-            await client.aclose()
-            return None
         except Exception as e:
-            console.print(f"[bold red][!][/bold red] Connection error during auth: {e}")
+            console.print(f"[bold red][!][/bold red] Connection error: {e}")
             await client.aclose()
             return None