Refactor dataclass fields in AuthContext and Credentials for improved privacy and representation

Author: iMicknl

pyoverkiz/auth/base.py

@@ -4,16 +4,16 @@
 
 import datetime
 from collections.abc import Mapping
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 from typing import Any, Protocol
 
 
 @dataclass(slots=True)
 class AuthContext:
     """Authentication context holding tokens and expiration."""
 
-    access_token: str | None = None
-    refresh_token: str | None = None
+    access_token: str | None = field(default=None, repr=False)
+    refresh_token: str | None = field(default=None, repr=False)
     expires_at: datetime.datetime | None = None
 
     def is_expired(self, *, skew_seconds: int = 5) -> bool:

pyoverkiz/auth/credentials.py

@@ -2,7 +2,7 @@
 
 from __future__ import annotations
 
-from dataclasses import dataclass
+from dataclasses import dataclass, field
 
 
 class Credentials:
@@ -14,14 +14,14 @@ class UsernamePasswordCredentials(Credentials):
     """Credentials using username and password."""
 
     username: str
-    password: str
+    password: str = field(repr=False)
 
 
 @dataclass(slots=True)
 class TokenCredentials(Credentials):
     """Credentials using an (API) token."""
 
-    token: str
+    token: str = field(repr=False)
 
 
 @dataclass(slots=True)
@@ -33,5 +33,5 @@ class LocalTokenCredentials(TokenCredentials):
 class RexelOAuthCodeCredentials(Credentials):
     """Credentials using Rexel OAuth2 authorization code."""
 
-    code: str
+    code: str = field(repr=False)
     redirect_uri: str

pyoverkiz/models.py

@@ -475,7 +475,7 @@ def select_first_attribute_value(self, attributes: list[str]) -> StateType | Non
 class Action:
     """An action consists of multiple commands related to a single device, identified by its device URL."""
 
-    device_url: str
+    device_url: str = field(repr=obfuscate_id)
     commands: list[Command] = field(factory=list)
 
     def to_payload(self) -> dict[str, object]: