usermanager: Improve help on invalid arguments

api/main.py

@@ -46,7 +46,7 @@
 from pydantic import BaseModel
 from jose import jwt
 from jose.exceptions import JWTError
 from kernelci.api.models import (
     Node,
     Hierarchy,
     PublishEvent,
@@ -895,7 +895,13 @@
 
     if from_ts:
         if isinstance(from_ts, str):
-            from_ts = datetime.fromisoformat(from_ts)
+            try:
+                from_ts = datetime.fromisoformat(from_ts)
+            except ValueError as exc:
+                raise HTTPException(
+                    status_code=status.HTTP_400_BAD_REQUEST,
+                    detail="Invalid 'from' parameter, must be an ISO 8601 datetime"
+                ) from exc
         query_params['timestamp'] = {'$gt': from_ts}
     if path:
         query_params['data.path'] = {'$regex': path}

scripts/usermanager.py

@@ -1,4 +1,4 @@
 #!/usr/bin/env python3
 import argparse
 import getpass
 import json
@@ -133,7 +133,7 @@
         raise SystemExit(1)
     try:
         payload = json.loads(body) if body else []
     except json.JSONDecodeError as exc:
         raise SystemExit("Failed to parse users response") from exc
     items = _parse_paginated_items(payload)
     matches = []
@@ -174,7 +174,7 @@
         raise SystemExit(1)
     try:
         payload = json.loads(body) if body else {}
     except json.JSONDecodeError as exc:
         raise SystemExit("Failed to parse user-groups response") from exc
     items = _parse_paginated_items(payload)
     matches = [
@@ -203,7 +203,7 @@
         raise SystemExit(1)
     try:
         payload = json.loads(body) if body else {}
     except json.JSONDecodeError as exc:
         raise SystemExit("Failed to parse user-group response") from exc
     resolved_name = payload.get("name")
     if not resolved_name:
@@ -222,7 +222,7 @@
         raise SystemExit(1)
     try:
         payload = json.loads(body) if body else {}
     except json.JSONDecodeError as exc:
         raise SystemExit("Failed to parse user response") from exc
     current_groups = _extract_group_names(payload)
     data = {
@@ -248,7 +248,7 @@
         with urllib.request.urlopen(req) as response:
             payload = response.read().decode("utf-8")
             return response.status, payload
     except urllib.error.HTTPError as exc:
         payload = exc.read().decode("utf-8")
         return exc.code, payload
 
@@ -274,7 +274,7 @@
     )
 
 
 def main():
     command_help = [
         ("accept-invite", "Accept an invite"),
         ("assign-group", "Assign group(s) to a user"),
@@ -298,14 +298,9 @@
     default_paths = "\n".join(f"  - {path}" for path in DEFAULT_CONFIG_PATHS)
     parser = argparse.ArgumentParser(
         description="KernelCI API user management helper",
-        usage=(
-            "usermanager.py [-h] [--config CONFIG] [--api-url API_URL] "
-            "[--token TOKEN] [--instance INSTANCE] [--token-label TOKEN_LABEL]\n"
-            "                <command> [<args>]\n\n"
-            "Commands:\n"
-            f"{command_list}"
-        ),
         epilog=(
+            "Commands:\n"
+            f"{command_list}\n\n"
             "Examples:\n"
             "  ./scripts/usermanager.py invite --username alice --email "
             "alice@example.org --return-token\n"
@@ -575,7 +570,7 @@
         if status < 400:
             try:
                 payload = json.loads(body) if body else {}
             except json.JSONDecodeError as exc:
                 raise SystemExit("Failed to parse login response") from exc
             token = payload.get("access_token")
             if not token: