Make (error) response on /openeo/x.y/.well-known/openeo more cache-friendly

give error body a static request id (just based on current request) to avoid confusion

ref: eu-cdse/openeo-cdse-infra#750

Author: soxofaan

openeo_driver/testing.py

@@ -123,7 +123,7 @@ def text(self) -> str:
     def headers(self) -> Headers:
         return self.response.headers
 
-    def assert_status_code(self, status_code: int) -> 'ApiResponse':
+    def assert_http_status_code(self, status_code: int) -> "ApiResponse":
         """Check HTTP status code"""
         if self.status_code != status_code:
             message = f"Expected response with status code {status_code} but got {self.status_code}."
@@ -132,6 +132,8 @@ def assert_status_code(self, status_code: int) -> 'ApiResponse':
             raise ApiException(message)
         return self
 
+    assert_status_code = assert_http_status_code
+
     def assert_content(self) -> 'ApiResponse':
         """Assert that the response body is not empty"""
         # TODO: also check content type? also check (prefix of) data?

openeo_driver/views.py

@@ -524,7 +524,10 @@ def udf_runtimes():
     @blueprint.route('/.well-known/openeo')
     def versioned_well_known_openeo():
         # Clients might request this for version discovery. Avoid polluting (error) logs by explicitly handling this.
-        error = NotFoundException(message="Not a well-known openEO URI")
+        error = NotFoundException(
+            message="Not a well-known openEO URI",
+            id="r-" + re.sub("[^a-z0-9]+", "", flask.request.path, flags=re.I),
+        )
         return make_response(jsonify(error.to_dict()), error.status_code)
 
     @blueprint.route('/CHANGELOG', methods=['GET'])

tests/test_views.py

@@ -191,7 +191,14 @@ def test_well_known_openeo(self, client):
         assert resp.headers["Cache-Control"] == "max-age=900, public"
 
     def test_versioned_well_known_openeo(self, api):
-        api.get("/.well-known/openeo").assert_error(404, "NotFound")
+        resp = api.get("/.well-known/openeo")
+        resp.assert_http_status_code(404)
+        short_api_version = api.api_version.replace(".", "")
+        assert resp.json == {
+            "code": "NotFound",
+            "id": f"r-openeo{short_api_version}wellknownopeneo",
+            "message": "Not a well-known openEO URI",
+        }
 
     @pytest.mark.parametrize(
         ["headers", "expected"],