eodag 3.4.2__tar.gz → 3.4.3__tar.gz

{eodag-3.4.2 → eodag-3.4.3}/.github/pull_request_template.md

@@ -1,5 +1,5 @@
 ### Your checklist for this pull request
-Please review the [guidelines for contributing](../CONTRIBUTING.md) to this repository.
+Please review the [guidelines for contributing](https://eodag.readthedocs.io/en/latest/contribute.html) to this repository.
 
 - [ ] Make sure you are making a pull request against the **develop branch** (left side). Also you should start *your branch* off *our develop*.
 - [ ] Check the commit's or even all commits' message styles matches our requested structure.

{eodag-3.4.2 → eodag-3.4.3}/CHANGES.rst

@@ -3,6 +3,63 @@ Release history
 ===============
 
 
+v3.4.3 (2025-06-12)
+===================
+
+Bug Fixes
+---------
+
+* **core**: Queryables mismatch when list of possible values contains a single value (`#1666`_,
+  `538331d`_)
+
+* **plugins**: GenericAuth missing credentials handle (`#1678`_, `576a2ac`_)
+
+* **plugins**: Openid_connect requests error handling (#1320) (`#1663`_, `9926083`_)
+
+* **plugins**: Order retry (`#1676`_, `3602426`_)
+
+* **providers**: Dedl mapping for CORINE collection (`#1661`_, `4c61b54`_)
+
+* **providers**: Wekeo_main orderable products download (`#1670`_, `d573846`_)
+
+Chores
+------
+
+* **deploy**: Remove deprecated common values (`154ea6d`_)
+
+Documentation
+-------------
+
+* Configuration environment variables defaults (`#1681`_, `6e8eb6b`_)
+
+* Updated contribution guidelines link in PR template (`#1667`_, `e5cd082`_)
+
+Refactoring
+-----------
+
+* Typing fixes following mypy 1.16.0 (`#1673`_, `ece52c0`_)
+
+.. _#1661: https://github.com/CS-SI/eodag/pull/1661
+.. _#1663: https://github.com/CS-SI/eodag/pull/1663
+.. _#1666: https://github.com/CS-SI/eodag/pull/1666
+.. _#1667: https://github.com/CS-SI/eodag/pull/1667
+.. _#1670: https://github.com/CS-SI/eodag/pull/1670
+.. _#1673: https://github.com/CS-SI/eodag/pull/1673
+.. _#1676: https://github.com/CS-SI/eodag/pull/1676
+.. _#1678: https://github.com/CS-SI/eodag/pull/1678
+.. _#1681: https://github.com/CS-SI/eodag/pull/1681
+.. _154ea6d: https://github.com/CS-SI/eodag/commit/154ea6d035572e64c3a434bb41c095c9b4cc76b2
+.. _3602426: https://github.com/CS-SI/eodag/commit/360242653ddc2a5c8587b37b3d91800459f4c243
+.. _4c61b54: https://github.com/CS-SI/eodag/commit/4c61b540ee46a8ae70932d64e9d373653763eb16
+.. _538331d: https://github.com/CS-SI/eodag/commit/538331d30085a814307173913ff831ca5a3397af
+.. _576a2ac: https://github.com/CS-SI/eodag/commit/576a2ac95044d10367e91e5ef843fb33a921f5f5
+.. _6e8eb6b: https://github.com/CS-SI/eodag/commit/6e8eb6b94eaad6294fea45d764a0e7c18a4e6823
+.. _9926083: https://github.com/CS-SI/eodag/commit/99260837837c3b5f2eeac8b95dc2b2feae7a0390
+.. _d573846: https://github.com/CS-SI/eodag/commit/d5738465930e08b24d562af3b7bc040464ff970a
+.. _e5cd082: https://github.com/CS-SI/eodag/commit/e5cd082aa81eedb62cd48b7974362c99a6899d9c
+.. _ece52c0: https://github.com/CS-SI/eodag/commit/ece52c07685e5df21cfda0b6ddc6a7416194406c
+
+
 v3.4.2 (2025-05-15)
 ===================
 

{eodag-3.4.2/eodag.egg-info → eodag-3.4.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: eodag
-Version: 3.4.2
+Version: 3.4.3
 Summary: Earth Observation Data Access Gateway
 Home-page: https://github.com/CS-SI/eodag
 Author: CS GROUP - France
@@ -317,7 +317,7 @@ An eodag instance can be exposed through a STAC compliant REST api from the comm
 
 .. code-block:: bash
 
-    docker run -p 5000:5000 --rm csspace/eodag-server:3.4.2
+    docker run -p 5000:5000 --rm csspace/eodag-server:3.4.3
 
 You can also browse over your STAC API server using `STAC Browser <https://github.com/radiantearth/stac-browser>`_.
 Simply run:

{eodag-3.4.2 → eodag-3.4.3}/README.rst

@@ -187,7 +187,7 @@ An eodag instance can be exposed through a STAC compliant REST api from the comm
 
 .. code-block:: bash
 
-    docker run -p 5000:5000 --rm csspace/eodag-server:3.4.2
+    docker run -p 5000:5000 --rm csspace/eodag-server:3.4.3
 
 You can also browse over your STAC API server using `STAC Browser <https://github.com/radiantearth/stac-browser>`_.
 Simply run:

{eodag-3.4.2 → eodag-3.4.3}/charts/eodag-server/Chart.yaml

@@ -1,5 +1,5 @@
 apiVersion: v2
-appVersion: 3.4.2
+appVersion: 3.4.3
 dependencies:
 - name: common
   repository:  oci://registry-1.docker.io/bitnamicharts
@@ -15,4 +15,4 @@ name: eodag-server
 sources:
 - https://github.com/CS-SI/eodag
 type: application
-version: 3.4.2
+version: 3.4.3

{eodag-3.4.2 → eodag-3.4.3}/charts/eodag-server/templates/ingress.yaml

@@ -18,7 +18,7 @@ metadata:
     {{- end }}
   {{- end }}
 spec:
-  {{- if and .Values.ingress.ingressClassName (eq "true" (include "common.ingress.supportsIngressClassname" .)) }}
+  {{- if .Values.ingress.ingressClassName }}
   ingressClassName: {{ .Values.ingress.ingressClassName | quote }}
   {{- end }}
   rules:
@@ -30,9 +30,7 @@ spec:
           {{- toYaml .Values.ingress.extraPaths | nindent 10 }}
           {{- end }}
           - path: {{ .Values.ingress.path }}
-            {{- if eq "true" (include "common.ingress.supportsPathType" .) }}
             pathType: {{ .Values.ingress.pathType }}
-            {{- end }}
             backend: {{- include "common.ingress.backend" (dict "serviceName" (include "common.names.fullname" .) "servicePort" .Values.ingress.servicePort "context" $)  | nindent 14 }}
     {{- end }}
     {{- range .Values.ingress.extraHosts }}
@@ -40,9 +38,7 @@ spec:
       http:
         paths:
           - path: {{ default "/" .path }}
-            {{- if eq "true" (include "common.ingress.supportsPathType" $) }}
             pathType: {{ default "ImplementationSpecific" .pathType }}
-            {{- end }}
             backend: {{- include "common.ingress.backend" (dict "serviceName" (include "common.names.fullname" $) "servicePort" .Values.ingress.servicePort "context" $) | nindent 14 }}
     {{- end }}
     {{- if .Values.ingress.extraRules }}

{eodag-3.4.2 → eodag-3.4.3}/docs/getting_started_guide/configure.rst

@@ -142,16 +142,21 @@ Core configuration using environment variables
 
 Some EODAG core settings can be overriden using environment variables:
 
-* ``EODAG_CFG_DIR`` customized configuration directory in place of ``~/.config/eodag``
+* ``EODAG_CFG_DIR`` customized configuration directory in place of `~/.config/eodag`.
 * ``EODAG_CFG_FILE`` for defining the desired path to the `user configuration file\
   <https://eodag.readthedocs.io/en/stable/getting_started_guide/configure.html#yaml-user-configuration-file>`_
+  in place of `~/.config/eodag/eodag.yml`.
 * ``EODAG_LOCS_CFG_FILE`` for defining the desired path to the
   `locations <https://eodag.readthedocs.io/en/stable/notebooks/api_user_guide/4_search.html#Locations-search>`_
-  configuration file
-* ``EODAG_PROVIDERS_CFG_FILE`` for defining the desired path to the providers configuration file
-* ``EODAG_PRODUCT_TYPES_CFG_FILE`` for defining the desired path to the product types configuration file
+  configuration file in place of `~/.config/eodag/locations.yml`.
+* ``EODAG_PROVIDERS_CFG_FILE`` for defining the desired path to the providers configuration file in place of
+  `<python-site-packages>/eodag/resources/providers.yml`.
+* ``EODAG_PRODUCT_TYPES_CFG_FILE`` for defining the desired path to the product types configuration file in place of
+  `<python-site-packages>/eodag/resources/product_types.yml`.
 * ``EODAG_EXT_PRODUCT_TYPES_CFG_FILE`` for defining the desired path to the `external product types configuration file\
   <https://eodag.readthedocs.io/en/stable/notebooks/api_user_guide/2_providers_products_available.html#Product-types-discovery>`_
+  in place of https://cs-si.github.io/eodag/eodag/resources/ext_product_types.json.
+  If the file is not readable, only user-modified providers will be fetched.
 
 CLI configuration
 ^^^^^^^^^^^^^^^^^

{eodag-3.4.2 → eodag-3.4.3}/docs/notebooks/api_user_guide/2_providers_products_available.ipynb

@@ -702,7 +702,7 @@
     "\n",
     "In EODAG, the discovered *EODAG external product types configuration file* can be set to:\n",
     "\n",
-    "* a file automatically built from github actions and stored in [eodag/resources/ext_product_types.json](https://raw.githubusercontent.com/CS-SI/eodag/develop/eodag/resources/ext_product_types.json) (default settings)\n",
+    "* a file automatically built from github actions and stored in [eodag/resources/ext_product_types.json](https://cs-si.github.io/eodag/eodag/resources/ext_product_types.json) (default settings)\n",
     "* a custom remote or local file by setting its path in `EODAG_EXT_PRODUCT_TYPES_CFG_FILE` environment variable (if the file is not readable, only user-modified providers will be fetched).\n",
     "\n",
     "Then, when listing product types using [list_product_types(fetch_providers=True)](../../api_reference/core.rst#eodag.api.core.EODataAccessGateway.list_product_types), EODAG will first read the content of the *EODAG external product types configuration file* using [fetch_product_types_list()](../../api_reference/core.rst#eodag.api.core.EODataAccessGateway.fetch_product_types_list) \n",

{eodag-3.4.2 → eodag-3.4.3}/docs/stac_rest.rst

@@ -115,7 +115,7 @@ available on `https://hub.docker.com/r/csspace/eodag-server <https://hub.docker.
 
 .. code-block:: bash
 
-    $ docker run -p 5000:5000 --rm csspace/eodag-server:3.4.2
+    $ docker run -p 5000:5000 --rm csspace/eodag-server:3.4.3
 
 Example
 -------

{eodag-3.4.2 → eodag-3.4.3}/eodag/api/core.py

@@ -658,7 +658,7 @@ class EODataAccessGateway:
             discovery_conf = getattr(
                 provider_search_config, "discover_product_types", {}
             )
-            if discovery_conf.get("fetch_url", None):
+            if discovery_conf.get("fetch_url"):
                 providers_discovery_configs_fetchable[
                     provider_to_fetch
                 ] = discovery_conf
@@ -744,7 +744,7 @@ class EODataAccessGateway:
                     user_discovery_conf == default_discovery_conf
                     or user_discovery_conf == default_discovery_conf_parsed
                 ) and (
-                    not default_discovery_conf.get("fetch_url", None)
+                    not default_discovery_conf.get("fetch_url")
                     or "ext_product_types_conf" not in locals()
                     or "ext_product_types_conf" in locals()
                     and (
@@ -856,7 +856,7 @@ class EODataAccessGateway:
                         continue
                     if not getattr(
                         search_plugin_config, "discover_product_types", {}
-                    ).get("fetch_url", None):
+                    ).get("fetch_url"):
                         # conf has been updated and provider product types are no more discoverable
                         continue
                     provider_products_config = (
@@ -980,7 +980,7 @@ class EODataAccessGateway:
         product_types = [
             k
             for k, v in self.product_types_config.items()
-            if v.get("alias", None) == alias_or_id
+            if v.get("alias") == alias_or_id
         ]
 
         if len(product_types) > 1:
@@ -1310,8 +1310,8 @@ class EODataAccessGateway:
         # since it might be modified if the next_page_url mechanism is used by the
         # plugin. (same thing for next_page_query_obj, next_page_query_obj with POST reqs)
         pagination_config = getattr(search_plugin.config, "pagination", {})
-        prev_next_page_url_tpl = pagination_config.get("next_page_url_tpl", None)
-        prev_next_page_query_obj = pagination_config.get("next_page_query_obj", None)
+        prev_next_page_url_tpl = pagination_config.get("next_page_url_tpl")
+        prev_next_page_query_obj = pagination_config.get("next_page_query_obj")
         # Page has to be set to a value even if use_next is True, this is required
         # internally by the search plugin (see collect_search_urls)
         kwargs.update(
@@ -1549,7 +1549,7 @@ class EODataAccessGateway:
         :param kwargs: Search criteria to help finding the right product
         :returns: A search result with one EO product or None at all
         """
-        product_type = kwargs.get("productType", None)
+        product_type = kwargs.get("productType")
         if product_type is not None:
             try:
                 product_type = self.get_product_type_from_alias(product_type)
@@ -1686,7 +1686,7 @@ class EODataAccessGateway:
                        * other criteria compatible with the provider
         :returns: Search plugins list and the prepared kwargs to make a query.
         """
-        product_type = kwargs.get("productType", None)
+        product_type: Optional[str] = kwargs.get("productType")
         if product_type is None:
             try:
                 guesses = self.guess_product_type(**kwargs)
@@ -1706,7 +1706,7 @@ class EODataAccessGateway:
                 # By now, only use the best bet
                 product_type = guesses[0]
             except NoMatchingProductType:
-                queried_id = kwargs.get("id", None)
+                queried_id = kwargs.get("id")
                 if queried_id is None:
                     logger.info(
                         "No product type could be guessed with provided arguments"
@@ -1750,7 +1750,7 @@ class EODataAccessGateway:
             product_type
             not in self._plugins_manager.product_type_to_provider_config_map.keys()
         ):
-            if provider:
+            if provider and product_type:
                 # Try to get specific product type from external provider
                 logger.debug(f"Fetching {provider} to find {product_type} product type")
                 self._fetch_external_product_type(provider, product_type)
@@ -1795,7 +1795,8 @@ class EODataAccessGateway:
         # Add product_types_config to plugin config. This dict contains product
         # type metadata that will also be stored in each product's properties.
         for search_plugin in search_plugins:
-            self._attach_product_type_config(search_plugin, product_type)
+            if product_type is not None:
+                self._attach_product_type_config(search_plugin, product_type)
 
         return search_plugins, kwargs
 

{eodag-3.4.2 → eodag-3.4.3}/eodag/api/product/_product.py

@@ -143,7 +143,7 @@ class EOProduct:
             and "defaultGeometry" not in properties
         ):
             raise MisconfiguredError(
-                f"No geometry available to build EOProduct(id={properties.get('id', None)}, provider={provider})"
+                f"No geometry available to build EOProduct(id={properties.get('id')}, provider={provider})"
             )
         elif not properties["geometry"] or properties["geometry"] == NOT_AVAILABLE:
             product_geometry = properties.pop("defaultGeometry", DEFAULT_GEOMETRY)
@@ -445,7 +445,7 @@ class EOProduct:
                     }
                 )
 
-        if self.properties.get("quicklook", None) is None:
+        if self.properties.get("quicklook") is None:
             logger.warning(
                 "Missing information to retrieve quicklook for EO product: %s",
                 self.properties["id"],
@@ -555,7 +555,7 @@ class EOProduct:
         return NoDriver()
 
     def _repr_html_(self):
-        thumbnail = self.properties.get("thumbnail", None)
+        thumbnail = self.properties.get("thumbnail")
         thumbnail_html = (
             f"<img src='{thumbnail}' width=100 alt='thumbnail'/>"
             if thumbnail and not thumbnail.startswith("s3")
@@ -576,12 +576,12 @@ class EOProduct:
                         {dict_to_html_table({
                          "provider": self.provider,
                          "product_type": self.product_type,
-                         "properties[&quot;id&quot;]": self.properties.get('id', None),
+                         "properties[&quot;id&quot;]": self.properties.get('id'),
                          "properties[&quot;startTimeFromAscendingNode&quot;]": self.properties.get(
-                             'startTimeFromAscendingNode', None
+                             'startTimeFromAscendingNode'
                          ),
                          "properties[&quot;completionTimeFromAscendingNode&quot;]": self.properties.get(
-                             'completionTimeFromAscendingNode', None
+                             'completionTimeFromAscendingNode'
                          ),
                          }, brackets=False)}
                         <details><summary style='color: grey; margin-top: 10px;'>properties:&ensp;({len(

{eodag-3.4.2 → eodag-3.4.3}/eodag/api/product/metadata_mapping.py

@@ -424,7 +424,7 @@ def format_metadata(search_param: str, *args: Any, **kwargs: Any) -> str:
                 return Polygon(polygon_args)
             elif len(georss) == 1 and "multisurface" in georss[0].tag.lower():
                 # Multipolygon
-                from_proj = getattr(georss[0], "attrib", {}).get("srsName", None)
+                from_proj = getattr(georss[0], "attrib", {}).get("srsName")
                 if from_proj:
                     from_proj = pyproj.CRS(from_proj)
                     to_proj = pyproj.CRS(DEFAULT_PROJ)
@@ -1030,8 +1030,8 @@ def properties_from_json(
     if not discovery_config:
         discovery_config = {}
 
-    discovery_pattern = discovery_config.get("metadata_pattern", None)
-    discovery_path = discovery_config.get("metadata_path", None)
+    discovery_pattern = discovery_config.get("metadata_pattern")
+    discovery_path = discovery_config.get("metadata_path")
     if discovery_pattern and discovery_path:
         discovery_jsonpath = string_to_jsonpath(discovery_path)
         discovered_properties = (
@@ -1215,8 +1215,8 @@ def properties_from_xml(
     # adds missing discovered properties
     if not discovery_config:
         discovery_config = {}
-    discovery_pattern = discovery_config.get("metadata_pattern", None)
-    discovery_path = discovery_config.get("metadata_path", None)
+    discovery_pattern = discovery_config.get("metadata_pattern")
+    discovery_path = discovery_config.get("metadata_path")
     if discovery_pattern and discovery_path:
         discovered_properties = root.xpath(
             discovery_path,
@@ -1524,7 +1524,7 @@ def get_provider_queryable_path(
     :param metadata_mapping: metadata-mapping configuration
     :returns: EODAG configured queryable path or None
     """
-    parameter_conf = metadata_mapping.get(queryable, None)
+    parameter_conf = metadata_mapping.get(queryable)
     if isinstance(parameter_conf, list):
         return parameter_conf[0]
     else:

{eodag-3.4.2 → eodag-3.4.3}/eodag/config.py

@@ -851,7 +851,7 @@ def override_config_from_env(config: dict[str, Any]) -> None:
         iter_parts = iter(parts)
         env_type = get_type_hints(PluginConfig).get(next(iter_parts, ""), str)
         child_env_type = (
-            get_type_hints(env_type).get(next(iter_parts, ""), None)
+            get_type_hints(env_type).get(next(iter_parts, ""))
             if isclass(env_type)
             else None
         )
@@ -961,7 +961,7 @@ def merge_configs(config: dict[str, Any], other_config: dict[str, Any]) -> None:
     other_config = dict(config, **other_config)
 
     for provider, new_conf in other_config.items():
-        old_conf = config.get(provider, None)
+        old_conf = config.get(provider)
 
         if old_conf:
             # update non-objects values

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/apis/ecmwf.py

@@ -123,16 +123,14 @@ class EcmwfApi(Api, ECMWFSearch):
         # start date
         if "startTimeFromAscendingNode" not in kwargs:
             kwargs["startTimeFromAscendingNode"] = (
-                getattr(self.config, "product_type_config", {}).get(
-                    "missionStartDate", None
-                )
+                getattr(self.config, "product_type_config", {}).get("missionStartDate")
                 or DEFAULT_MISSION_START_DATE
             )
         # end date
         if "completionTimeFromAscendingNode" not in kwargs:
             kwargs["completionTimeFromAscendingNode"] = getattr(
                 self.config, "product_type_config", {}
-            ).get("missionEndDate", None) or datetime.now(timezone.utc).isoformat(
+            ).get("missionEndDate") or datetime.now(timezone.utc).isoformat(
                 timespec="seconds"
             )
 
@@ -149,8 +147,8 @@ class EcmwfApi(Api, ECMWFSearch):
         :raises: :class:`~eodag.utils.exceptions.AuthenticationError`
         """
         # Get credentials from eodag or using ecmwf conf
-        email = getattr(self.config, "credentials", {}).get("username", None)
-        key = getattr(self.config, "credentials", {}).get("password", None)
+        email = getattr(self.config, "credentials", {}).get("username")
+        key = getattr(self.config, "credentials", {}).get("password")
         url = getattr(self.config, "auth_endpoint", None)
         if not all([email, key, url]):
             key, url, email = get_apikey_values()
@@ -295,5 +293,5 @@ class EcmwfApi(Api, ECMWFSearch):
                        arguments)
         :returns: fetched queryable parameters dict
         """
-        product_type = kwargs.get("productType", None)
+        product_type = kwargs.get("productType")
         return self.queryables_from_metadata_mapping(product_type)

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/apis/usgs.py

@@ -241,7 +241,7 @@ class UsgsApi(Api):
             download_options = api.download_options(
                 usgs_dataset, list(results_by_entity_id.keys())
             )
-            if download_options.get("data", None) is not None:
+            if download_options.get("data") is not None:
                 for download_option in download_options["data"]:
                     # update results with available downloadSystem
                     if (

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/authentication/generic.py

@@ -46,7 +46,11 @@ class GenericAuth(Authentication):
         """Authenticate"""
         self.validate_config_credentials()
         method = getattr(self.config, "method", "basic")
-
+        if not all(x in self.config.credentials for x in ["username", "password"]):
+            raise MisconfiguredError(
+                f"Missing credentials for provider {self.provider}",
+                "You must provide 'username' and 'password' in the configuration.",
+            )
         if method == "digest":
             return HTTPDigestAuth(
                 self.config.credentials["username"],

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/authentication/openid_connect.py

@@ -31,7 +31,12 @@ from requests.auth import AuthBase
 
 from eodag.plugins.authentication import Authentication
 from eodag.utils import HTTP_REQ_TIMEOUT, USER_AGENT, parse_qs, repeatfunc, urlparse
-from eodag.utils.exceptions import AuthenticationError, MisconfiguredError, TimeOutError
+from eodag.utils.exceptions import (
+    AuthenticationError,
+    MisconfiguredError,
+    RequestError,
+    TimeOutError,
+)
 
 if TYPE_CHECKING:
     from requests import PreparedRequest, Response
@@ -304,12 +309,17 @@ class OIDCAuthorizationCodeFlowAuth(OIDCRefreshTokenBase):
         state = self.compute_state()
         authentication_response = self.authenticate_user(state)
         exchange_url = authentication_response.url
+
         for err_pattern, err_message in getattr(
             self.config, "exchange_url_error_pattern", {}
         ).items():
             if err_pattern in exchange_url:
                 raise AuthenticationError(err_message)
+
         if not exchange_url.startswith(self.config.redirect_uri):
+            if "Invalid username or password" in authentication_response.text:
+                raise AuthenticationError("Invalid username or password")
+
             raise AuthenticationError(
                 f"Could not authenticate user with provider {self.provider}.",
                 "Please verify your credentials",
@@ -369,13 +379,20 @@ class OIDCAuthorizationCodeFlowAuth(OIDCRefreshTokenBase):
             "redirect_uri": self.config.redirect_uri,
         }
         ssl_verify = getattr(self.config, "ssl_verify", True)
-        authorization_response = self.session.get(
-            self.authorization_endpoint,
-            params=params,
-            headers=USER_AGENT,
-            timeout=HTTP_REQ_TIMEOUT,
-            verify=ssl_verify,
-        )
+        try:
+            authorization_response = self.session.get(
+                self.authorization_endpoint,
+                params=params,
+                headers=USER_AGENT,
+                timeout=HTTP_REQ_TIMEOUT,
+                verify=ssl_verify,
+            )
+        except requests.exceptions.Timeout as exc:
+            raise TimeoutError(exc, "The authentication request timed out.") from exc
+        except requests.RequestException as exc:
+            raise RequestError.from_error(
+                exc, "An error occurred while authenticating the user."
+            ) from exc
 
         login_document = etree.HTML(authorization_response.text)
         login_forms = login_document.xpath(self.config.login_form_xpath)
@@ -413,13 +430,20 @@ class OIDCAuthorizationCodeFlowAuth(OIDCRefreshTokenBase):
             auth_uri = getattr(self.config, "authentication_uri", None)
             if not auth_uri:
                 raise MisconfiguredError("authentication_uri is missing")
-        return self.session.post(
-            auth_uri,
-            data=login_data,
-            headers=USER_AGENT,
-            timeout=HTTP_REQ_TIMEOUT,
-            verify=ssl_verify,
-        )
+        try:
+            return self.session.post(
+                auth_uri,
+                data=login_data,
+                headers=USER_AGENT,
+                timeout=HTTP_REQ_TIMEOUT,
+                verify=ssl_verify,
+            )
+        except requests.exceptions.Timeout as exc:
+            raise TimeoutError(exc, "The authentication request timed out.") from exc
+        except requests.RequestException as exc:
+            raise RequestError.from_error(
+                exc, "An error occurred while authenticating the user."
+            ) from exc
 
     def grant_user_consent(self, authentication_response: Response) -> Response:
         """Grant user consent"""
@@ -433,13 +457,20 @@ class OIDCAuthorizationCodeFlowAuth(OIDCRefreshTokenBase):
             for key, value in self.config.user_consent_form_data.items()
         }
         ssl_verify = getattr(self.config, "ssl_verify", True)
-        return self.session.post(
-            self.authorization_endpoint,
-            data=user_consent_data,
-            headers=USER_AGENT,
-            timeout=HTTP_REQ_TIMEOUT,
-            verify=ssl_verify,
-        )
+        try:
+            return self.session.post(
+                self.authorization_endpoint,
+                data=user_consent_data,
+                headers=USER_AGENT,
+                timeout=HTTP_REQ_TIMEOUT,
+                verify=ssl_verify,
+            )
+        except requests.exceptions.Timeout as exc:
+            raise TimeoutError(exc, "The authentication request timed out.") from exc
+        except requests.RequestException as exc:
+            raise RequestError.from_error(
+                exc, "An error occurred while authenticating the user."
+            ) from exc
 
     def _prepare_token_post_data(self, token_data: dict[str, Any]) -> dict[str, Any]:
         """Prepare the common data to post to the token URI"""
@@ -487,14 +518,21 @@ class OIDCAuthorizationCodeFlowAuth(OIDCRefreshTokenBase):
             self.config.token_exchange_post_data_method: token_exchange_data
         }
         ssl_verify = getattr(self.config, "ssl_verify", True)
-        r = self.session.post(
-            self.token_endpoint,
-            headers=USER_AGENT,
-            timeout=HTTP_REQ_TIMEOUT,
-            verify=ssl_verify,
-            **post_request_kwargs,
-        )
-        return r
+        try:
+            r = self.session.post(
+                self.token_endpoint,
+                headers=USER_AGENT,
+                timeout=HTTP_REQ_TIMEOUT,
+                verify=ssl_verify,
+                **post_request_kwargs,
+            )
+            return r
+        except requests.exceptions.Timeout as exc:
+            raise TimeoutError(exc, "The authentication request timed out.") from exc
+        except requests.RequestException as exc:
+            raise RequestError.from_error(
+                exc, "An error occurred while authenticating the user."
+            ) from exc
 
     def _constant_or_xpath_extracted(
         self, value: str, form_element: Any

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/authentication/sas_auth.py

@@ -116,7 +116,7 @@ class SASAuth(Authentication):
         headers = deepcopy(USER_AGENT)
 
         # update headers with subscription key if exists
-        apikey = getattr(self.config, "credentials", {}).get("apikey", None)
+        apikey = getattr(self.config, "credentials", {}).get("apikey")
         ssl_verify = getattr(self.config, "ssl_verify", True)
         if apikey:
             headers_update = format_dict_items(self.config.headers, apikey=apikey)

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/crunch/filter_date.py

@@ -69,7 +69,7 @@ class FilterDate(Crunch):
             return []
 
         # filter start date
-        filter_start_str = self.config.__dict__.get("start", None)
+        filter_start_str = self.config.__dict__.get("start")
         if filter_start_str:
             filter_start = dateutil.parser.parse(filter_start_str)
             if not filter_start.tzinfo:
@@ -78,7 +78,7 @@ class FilterDate(Crunch):
             filter_start = None
 
         # filter end date
-        filter_end_str = self.config.__dict__.get("end", None)
+        filter_end_str = self.config.__dict__.get("end")
         if filter_end_str:
             filter_end = dateutil.parser.parse(filter_end_str)
             if not filter_end.tzinfo:
@@ -93,9 +93,7 @@ class FilterDate(Crunch):
         for product in products:
 
             # product start date
-            product_start_str = product.properties.get(
-                "startTimeFromAscendingNode", None
-            )
+            product_start_str = product.properties.get("startTimeFromAscendingNode")
             if product_start_str:
                 product_start = dateutil.parser.parse(product_start_str)
                 if not product_start.tzinfo:
@@ -104,9 +102,7 @@ class FilterDate(Crunch):
                 product_start = None
 
             # product end date
-            product_end_str = product.properties.get(
-                "completionTimeFromAscendingNode", None
-            )
+            product_end_str = product.properties.get("completionTimeFromAscendingNode")
             if product_end_str:
                 product_end = dateutil.parser.parse(product_end_str)
                 if not product_end.tzinfo:

{eodag-3.4.2 → eodag-3.4.3}/eodag/plugins/crunch/filter_property.py

@@ -64,7 +64,7 @@ class FilterProperty(Crunch):
             return products
 
         property_key = next(iter(self.config.__dict__))
-        property_value = self.config.__dict__.get(property_key, None)
+        property_value = self.config.__dict__.get(property_key)
 
         logger.debug(
             "Start filtering for products matching operator.%s(product.properties['%s'], %s)",