throw an error if stac api env variables aren't set, instead of resor…

…ting to authentication code flow #213

src/worldcereal/openeo/extract.py

@@ -139,11 +139,6 @@ def post_job_action_patch(
         username = os.getenv("STAC_API_USERNAME")
         password = os.getenv("STAC_API_PASSWORD")
 
-        if not username or not password:
-            pipeline_log.warning(
-                "STAC API credentials not found. Resorting to authorization code flow."
-            )
-
         stac_api_interaction = StacApiInteraction(
             sensor=sensor,
             base_url="https://stac.openeo.vito.be",

src/worldcereal/stac/stac_api_interaction.py

@@ -6,7 +6,6 @@
 import pystac_client
 import requests
 from openeo.rest.auth.oidc import (
-    OidcAuthCodePkceAuthenticator,
     OidcClientInfo,
     OidcProviderInfo,
     OidcResourceOwnerPasswordAuthenticator,
@@ -47,7 +46,9 @@ def get_access_token(self) -> str:
                 client_info=client_info, username=self.username, password=self.password
             )
         else:
-            authenticator = OidcAuthCodePkceAuthenticator(client_info=client_info)
+            raise ValueError(
+                "Credentials are required to obtain an access token. Please set STAC_API_USERNAME and STAC_API_PASSWORD environment variables."
+            )
 
         tokens = authenticator.get_tokens()