Skip to content

Commit 26d5c53

Browse files
authored
Merge pull request #207 from ecmwf-projects/COPDS-2041-warn-execute-deprecation
Add warning message for deprecated /execute endpoint
2 parents 4960972 + 57e4303 commit 26d5c53

File tree

2 files changed

+15
-0
lines changed

2 files changed

+15
-0
lines changed

cads_processing_api_service/clients.py

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,7 @@ def get_process(
177177

178178
def post_process_execution(
179179
self,
180+
api_request: fastapi.Request,
180181
process_id: str = fastapi.Path(...),
181182
execution_content: models.Execute = fastapi.Body(...),
182183
auth_header: tuple[str, str] = fastapi.Depends(auth.get_auth_header),
@@ -270,6 +271,14 @@ def post_process_execution(
270271
)
271272
for message in dataset.messages
272273
]
274+
url = str(api_request.url)
275+
if url.rstrip("/").endswith("execute"):
276+
message = models.DatasetMessage(
277+
date=datetime.datetime.now(),
278+
severity="WARNING",
279+
content=config.ensure_settings().deprecation_warning_message,
280+
)
281+
dataset_messages.append(message)
273282
status_info = utils.make_status_info(
274283
job, dataset_metadata={"messages": dataset_messages}
275284
)

cads_processing_api_service/config.py

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,11 @@
3434
"{licences}"
3535
)
3636

37+
DEPRECATION_WARNING_MESSAGE = (
38+
"You are using a deprecated API endpoint. "
39+
"If you are using cdsapi, please upgrade to the latest version."
40+
)
41+
3742
general_settings = None
3843

3944

@@ -57,6 +62,7 @@ class Settings(pydantic_settings.BaseSettings):
5762
api_request_template: str = API_REQUEST_TEMPLATE
5863
missing_dataset_title: str = "Dataset not available"
5964
anonymous_licences_message: str = ANONYMOUS_LICENCES_MESSAGE
65+
deprecation_warning_message: str = DEPRECATION_WARNING_MESSAGE
6066

6167
@property
6268
def profiles_api_url(self) -> str:

0 commit comments

Comments
 (0)