Log sync processing identifier by default #890

Author: soxofaan

CHANGELOG.md

@@ -10,6 +10,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 
 - Add public `split_area` function for tile-grid based job splitting
+- Log synchronous processing identifier ("OpenEO-Identifier" response header) by default (debug level) ([#890](https://github.com/Open-EO/openeo-python-client/issues/890))
+
 
 ### Changed
 

openeo/rest/connection.py

@@ -182,7 +182,8 @@ def __init__(
         self._refresh_token_store = refresh_token_store
         self._oidc_auth_renewer = oidc_auth_renewer
         self._auto_validate = auto_validate
-        self._on_response_headers_sync = on_response_headers_sync
+        if on_response_headers_sync:
+            self._on_response_headers_sync = on_response_headers_sync
 
     @classmethod
     def version_discovery(
@@ -222,6 +223,10 @@ def _get_auth_config(self) -> AuthConfig:
             self._auth_config = AuthConfig()
         return self._auth_config
 
+    def _on_response_headers_sync(self, headers: Mapping):
+        if identifier := headers.get("OpenEO-Identifier"):
+            _log.debug(f"Synchronous processing identifier: {identifier!r}")
+
     def _get_refresh_token_store(self) -> RefreshTokenStore:
         if self._refresh_token_store is None:
             self._refresh_token_store = RefreshTokenStore()

tests/rest/test_connection.py

@@ -4361,6 +4361,16 @@ def test_download_on_response_headers(dummy_backend, tmp_path):
     assert results == [{"OpenEO-Identifier": "r-001"}]
 
 
+def test_connection_on_response_headers_sync_default_download(dummy_backend, tmp_path, caplog):
+    connection = openeo.connect(dummy_backend.connection.root_url)
+    with caplog.at_level(logging.DEBUG, logger="openeo"):
+        connection.download(
+            {"foo1": {"process_id": "foo"}},
+            tmp_path / "result.data",
+        )
+    assert "Synchronous processing identifier: 'r-001'" in caplog.messages
+
+
 def test_connection_on_response_headers_sync_download(dummy_backend, tmp_path):
     results = []
     connection = openeo.connect(dummy_backend.connection.root_url, on_response_headers_sync=results.append)