python-openpublictransport 0.1.3__tar.gz → 0.1.4__tar.gz

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-openpublictransport
-Version: 0.1.3
+Version: 0.1.4
 Summary: Python library for public transport APIs (EFA, OTP2, TRIAS, FPTF, HAFAS, GTFS-RT)
 License: MIT
 Requires-Python: >=3.11

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-openpublictransport"
-version = "0.1.3"
+version = "0.1.4"
 description = "Python library for public transport APIs (EFA, OTP2, TRIAS, FPTF, HAFAS, GTFS-RT)"
 readme = "README.md"
 license = { text = "MIT" }

python_openpublictransport-0.1.4/src/openpublictransport/__init__.py

@@ -0,0 +1,12 @@
+"""python-openpublictransport — public transport API library."""
+
+from .exceptions import AuthenticationError
+from .providers import get_all_provider_ids, get_provider, get_provider_class, register_provider
+
+__all__ = [
+    "get_provider",
+    "get_provider_class",
+    "get_all_provider_ids",
+    "register_provider",
+    "AuthenticationError",
+]

python_openpublictransport-0.1.4/src/openpublictransport/exceptions.py

@@ -0,0 +1,9 @@
+"""Exceptions for python-openpublictransport."""
+
+
+class AuthenticationError(Exception):
+    """Raised when API authentication fails (HTTP 401 or 403).
+
+    Signals that the configured API key is invalid or expired.
+    Callers should prompt the user to re-enter credentials rather than retrying.
+    """

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/national_rail.py

@@ -14,6 +14,7 @@ from zoneinfo import ZoneInfo
 import aiohttp
 
 from ..const import PROVIDER_NATIONAL_RAIL
+from ..exceptions import AuthenticationError
 from ..models import UnifiedDeparture
 from .base import BaseProvider
 
@@ -118,6 +119,10 @@ class NationalRailProvider(BaseProvider):
                 },
                 timeout=aiohttp.ClientTimeout(total=15),
             ) as resp:
+                if resp.status in (401, 403):
+                    raise AuthenticationError(
+                        f"{self.provider_name}: authentication failed (HTTP {resp.status}) — check API key"
+                    )
                 if resp.status != 200:
                     _LOGGER.warning("%s: HTTP %s for CRS %s", self.provider_name, resp.status, crs)
                     return None

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/nta.py

@@ -6,6 +6,8 @@ from datetime import datetime, timedelta, timezone
 from typing import Any, Dict, List, Optional, Union
 from zoneinfo import ZoneInfo
 
+from ..exceptions import AuthenticationError
+
 import aiohttp
 from aiohttp import ClientConnectorError
 
@@ -217,13 +219,14 @@ class NTAProvider(BaseProvider):
                     elif response.status == 404:
                         _LOGGER.warning("NTA API endpoint not found (404)")
                         return None
-                    elif response.status == 401:
+                    elif response.status in (401, 403):
                         if self.api_key_secondary and current_api_key == self.api_key:
-                            _LOGGER.info("NTA Primary API key failed (401), trying Secondary key...")
+                            _LOGGER.info("NTA primary key failed (HTTP %s), trying secondary...", response.status)
                             current_api_key = self.api_key_secondary
                             continue
-                        _LOGGER.warning("NTA API authentication failed (401) - check API key(s)")
-                        return None
+                        raise AuthenticationError(
+                            f"NTA: authentication failed (HTTP {response.status}) — check API key(s)"
+                        )
                     elif response.status >= 500:
                         _LOGGER.warning(
                             "NTA API server error (status %s) on attempt %d/%d",

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/otp_base.py

@@ -3,6 +3,8 @@
 import asyncio
 import logging
 from datetime import datetime, timezone
+
+from ..exceptions import AuthenticationError
 from typing import Any, Dict, List, Optional, Tuple, Union
 from urllib.parse import quote
 from zoneinfo import ZoneInfo
@@ -86,6 +88,10 @@ class OTPBaseProvider(BaseProvider):
                     return await resp.json()
                 if resp.status == 204:
                     return None
+                if resp.status in (401, 403):
+                    raise AuthenticationError(
+                        f"{self.provider_name}: authentication failed (HTTP {resp.status}) — check API key"
+                    )
                 _LOGGER.warning("%s OTP %s → HTTP %s", self.provider_name, url, resp.status)
         except aiohttp.ClientError as exc:
             _LOGGER.warning("%s OTP request failed: %s", self.provider_name, exc)

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/rejseplanen.py

@@ -12,6 +12,7 @@ from zoneinfo import ZoneInfo
 import aiohttp
 
 from ..const import PROVIDER_REJSEPLANEN
+from ..exceptions import AuthenticationError
 from ..models import UnifiedDeparture
 from .base import BaseProvider
 
@@ -103,6 +104,10 @@ class RejseplanenProvider(BaseProvider):
         )
         try:
             async with self.session.get(url, timeout=aiohttp.ClientTimeout(total=15)) as resp:
+                if resp.status in (401, 403):
+                    raise AuthenticationError(
+                        f"{self.provider_name}: authentication failed (HTTP {resp.status}) — check API key"
+                    )
                 if resp.status != 200:
                     _LOGGER.warning("%s: HTTP %s for station %s", self.provider_name, resp.status, station_id)
                     return None

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/rmv.py

@@ -6,6 +6,8 @@ from typing import Any, Dict, List, Optional, Union
 from urllib.parse import quote
 from zoneinfo import ZoneInfo
 
+from ..exceptions import AuthenticationError
+
 import aiohttp
 
 from ..const import PROVIDER_RMV
@@ -126,8 +128,10 @@ class RMVProvider(BaseProvider):
                     if isinstance(departures, dict):
                         departures = [departures]
                     return {"stopEvents": departures}
-                elif response.status == 401:
-                    _LOGGER.error("RMV API: invalid API key")
+                elif response.status in (401, 403):
+                    raise AuthenticationError(
+                        f"RMV: authentication failed (HTTP {response.status}) — check API key"
+                    )
                 else:
                     _LOGGER.warning("RMV API returned status %s", response.status)
         except aiohttp.ClientError as e:
@@ -264,8 +268,14 @@ class RMVProvider(BaseProvider):
                             }
                         )
                     return results
+                elif response.status in (401, 403):
+                    raise AuthenticationError(
+                        f"RMV: authentication failed (HTTP {response.status}) — check API key"
+                    )
                 else:
                     _LOGGER.error("RMV API returned status %s", response.status)
+        except AuthenticationError:
+            raise
         except Exception as e:
             _LOGGER.error("Error searching RMV stops: %s", e)
 

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/trafiklab.py

@@ -7,6 +7,8 @@ from typing import Any, Dict, List, Optional, Union
 from urllib.parse import quote
 from zoneinfo import ZoneInfo
 
+from ..exceptions import AuthenticationError
+
 import aiohttp
 from aiohttp import ClientConnectorError
 
@@ -131,9 +133,10 @@ class TrafiklabProvider(BaseProvider):
                     elif response.status == 404:
                         _LOGGER.warning("Trafiklab API endpoint not found (404)")
                         return None
-                    elif response.status == 401:
-                        _LOGGER.warning("Trafiklab API authentication failed (401) - check API key")
-                        return None
+                    elif response.status in (401, 403):
+                        raise AuthenticationError(
+                            f"Trafiklab: authentication failed (HTTP {response.status}) — check API key"
+                        )
                     elif response.status >= 500:
                         _LOGGER.warning(
                             "Trafiklab API server error (status %s) on attempt %d/%d",
@@ -243,9 +246,10 @@ class TrafiklabProvider(BaseProvider):
                             results.append(result)
 
                         return results
-                    elif response.status == 401:
-                        _LOGGER.error("Trafiklab API authentication failed (401) - check API key")
-                        return []
+                    elif response.status in (401, 403):
+                        raise AuthenticationError(
+                            f"Trafiklab: authentication failed (HTTP {response.status}) — check API key"
+                        )
                     elif response.status == 404:
                         _LOGGER.warning("Trafiklab API endpoint not found (404)")
                         return []

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/openpublictransport/providers/trias_base.py

@@ -3,6 +3,8 @@
 import logging
 from datetime import datetime
 from typing import Any, Dict, List, Optional, Union
+
+from ..exceptions import AuthenticationError
 from xml.etree import ElementTree as ET
 from zoneinfo import ZoneInfo
 
@@ -140,6 +142,10 @@ class TRIASBaseProvider(BaseProvider):
                 if response.status == 200:
                     text = await response.text()
                     return ET.fromstring(text)
+                elif response.status in (401, 403):
+                    raise AuthenticationError(
+                        f"{self.provider_name}: authentication failed (HTTP {response.status}) — check API key"
+                    )
                 else:
                     _LOGGER.warning("%s TRIAS API returned status %s", self.provider_name, response.status)
         except aiohttp.ClientError as e:

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/python_openpublictransport.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-openpublictransport
-Version: 0.1.3
+Version: 0.1.4
 Summary: Python library for public transport APIs (EFA, OTP2, TRIAS, FPTF, HAFAS, GTFS-RT)
 License: MIT
 Requires-Python: >=3.11

{python_openpublictransport-0.1.3 → python_openpublictransport-0.1.4}/src/python_openpublictransport.egg-info/SOURCES.txt

@@ -1,6 +1,7 @@
 pyproject.toml
 src/openpublictransport/__init__.py
 src/openpublictransport/const.py
+src/openpublictransport/exceptions.py
 src/openpublictransport/models.py
 src/openpublictransport/parsers.py
 src/openpublictransport/providers/__init__.py

python_openpublictransport-0.1.3/src/openpublictransport/__init__.py

@@ -1,5 +0,0 @@
-"""python-openpublictransport — public transport API library."""
-
-from .providers import get_all_provider_ids, get_provider, get_provider_class, register_provider
-
-__all__ = ["get_provider", "get_provider_class", "get_all_provider_ids", "register_provider"]