From a8cf49e995c5a61ff0bc8cc7fa2d03ed13d27989 Mon Sep 17 00:00:00 2001 From: Cliff Hill Date: Wed, 28 Feb 2024 09:28:19 -0500 Subject: [PATCH] Fixed another error. Signed-off-by: Cliff Hill --- app/job/rest.py | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-) diff --git a/app/job/rest.py b/app/job/rest.py index 4893baa9d..3e239f14d 100644 --- a/app/job/rest.py +++ b/app/job/rest.py @@ -127,18 +127,30 @@ def get_jobs_by_service(service_id): except ValueError: errors = { "limit_days": [ - "{} is not an integer".format(request.args["limit_days"]) + f"{request.args['limit_days']} is not an integer" ] } raise InvalidRequest(errors, status_code=400) else: limit_days = None + valid_statuses = set(JobStatus) + statuses_arg = request.args.get("statuses", "") + if statuses_arg == "": + statuses = None + else: + statuses = [] + for x in statuses_arg.split(","): + status = x.strip() + if status in valid_statuses: + statuses.append(status) + else: + statuses.append(None) return jsonify( **get_paginated_jobs( service_id, limit_days=limit_days, - statuses=[x.strip() for x in request.args.get("statuses", "").split(",")], + statuses=statuses, page=int(request.args.get("page", 1)), ) )