Merge branch 'feature/zarr-output' into feature/lambda-docker

Open-EO · Oct 20, 2023 · a0b2b6a · a0b2b6a
2 parents 29fdcf1 + 249d3ac
commit a0b2b6a
Show file tree

Hide file tree

Showing 3 changed files with 76 additions and 5 deletions.
diff --git a/README.md b/README.md
@@ -39,11 +39,13 @@ Then the REST API server can be run:
 <pipenv> $ python app.py
 ```
 
-To locally run the REST API server on HTTPS, so you can use it in the [openEO editor](https://editor.openeo.org/), start it with:
+To locally run the REST API server on HTTPS, start it with:
 ```
 <pipenv> $ python app.py https
 ```
 
+To use it in the [openEO editor](https://editor.openeo.org/), you must first access the endpoint directly via a Browser and "Accept the Risk and Continue" (there is no valid security certificate behind this secure connection).
+
 ### Troubleshooting
 
 If validator complains about process graphs that are clearly correct (and which are valid on production deployment), there are two things than can be done:

diff --git a/rest/processing/processing_api_request.py b/rest/processing/processing_api_request.py
@@ -60,6 +60,10 @@ def make_request(self):
         return requests.post(self.url, data=json.dumps(self.data), headers=self.get_headers())
 
     def has_rate_limiting_with_backoff(self):
-        if self.url.startswith(SentinelhubDeployments.USWEST):
+        if (
+            self.url.startswith(SentinelhubDeployments.MAIN)
+            or self.url.startswith(SentinelhubDeployments.USWEST)
+            or self.url.startswith(SentinelhubDeployments.CREODIAS)
+        ):
             return True
         return False
diff --git a/tests/test_units.py b/tests/test_units.py
@@ -986,9 +986,40 @@ def test_get_roles(filename, expected_roles):
     "endpoint,access_token,api_responses,min_exec_time,should_raise_error,expected_error",
     [
         ("https://services.sentinel-hub.com", None, [{"status": 200}], 0, False, None),
-        ("https://services.sentinel-hub.com", None, [{"status": 429}], 0, True, "Too Many Requests"),
-        ("https://services.sentinel-hub.com", "some-token", [{"status": 200}], 0, False, None),
-        ("https://services.sentinel-hub.com", "some-token", [{"status": 429}], 0, True, "Too Many Requests"),
+        (
+            "https://services.sentinel-hub.com",
+            None,
+            [
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "7"}},
+                {"status": 200},
+            ],
+            9,
+            False,
+            None,
+        ),
+        (
+            "https://services.sentinel-hub.com",
+            None,
+            [
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "2"}},
+            ],
+            0,
+            True,
+            "Out of retries.",
+        ),
+        (
+            "https://services.sentinel-hub.com",
+            None,
+            [{"status": 429, "headers": {"retry-after": "2"}}, {"status": 500}],
+            0,
+            True,
+            "Internal Server Error",
+        ),
+        ("https://services-uswest2.sentinel-hub.com", None, [{"status": 200}], 0, False, None),
         (
             "https://services-uswest2.sentinel-hub.com",
             None,
@@ -1022,6 +1053,40 @@ def test_get_roles(filename, expected_roles):
             True,
             "Internal Server Error",
         ),
+        ("https://creodias.sentinel-hub.com", None, [{"status": 200}], 0, False, None),
+        (
+            "https://creodias.sentinel-hub.com",
+            None,
+            [
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "7"}},
+                {"status": 200},
+            ],
+            9,
+            False,
+            None,
+        ),
+        (
+            "https://creodias.sentinel-hub.com",
+            None,
+            [
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "2"}},
+                {"status": 429, "headers": {"retry-after": "2"}},
+            ],
+            0,
+            True,
+            "Out of retries.",
+        ),
+        (
+            "https://creodias.sentinel-hub.com",
+            None,
+            [{"status": 429, "headers": {"retry-after": "2"}}, {"status": 500}],
+            0,
+            True,
+            "Internal Server Error",
+        ),
     ],
 )
 def test_processing_api_request(