From 03843a6ba5d7751303954c20a7eab50e64544818 Mon Sep 17 00:00:00 2001
From: Karel van Klink <karel.vanklink@geant.org>
Date: Thu, 28 Dec 2023 10:11:35 +0100
Subject: [PATCH] use built-in raise_for_status in provisioning proxy service
---
gso/services/provisioning_proxy.py | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/gso/services/provisioning_proxy.py b/gso/services/provisioning_proxy.py
index 0aa6d8fb..d705f3de 100644
--- a/gso/services/provisioning_proxy.py
+++ b/gso/services/provisioning_proxy.py
@@ -6,7 +6,6 @@
import json
import logging
from functools import partial
-from http import HTTPStatus
from typing import Any
import requests
@@ -74,10 +73,7 @@ def _send_request(operation: CUDOperation, endpoint: str, parameters: dict, call
elif operation == CUDOperation.DELETE:
request = requests.delete(url, json=parameters, timeout=10000)
- if not HTTPStatus.OK <= request.status_code < HTTPStatus.MULTIPLE_CHOICES:
- # :term:`LSO` has returned a status code that is outside the 2xx range, and has therefore failed
- logger.debug(request.content)
- raise AssertionError(request.content)
+ request.raise_for_status()
_send_post = partial(_send_request, CUDOperation.POST)
--
GitLab