Mistral should returns 404 in the case of an absent execution's tasks request
Bug #1998210 reported by
Vasudeo Nimbekar
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Mistral |
Fix Released
|
Undecided
|
Vasudeo Nimbekar |
Bug Description
Mistral GET execution's tasks API returns 200 response status code and empty tasks response for absent execution id.
To reproduce:
- Execute GET request for v2/executions/
Ex:
request:
GET http://
response:
status code - 200
response data - {"tasks": []}
expected response:
status code - 404
response data -
{
"faultcode": "Client",
"faultstring": "WorkflowExecution not found [id=00000000-
"debuginfo": null
}
summary: |
- Mistral returns 200 OK in the case of an absent execution's tasks + Mistral should returns 404 in the case of an absent execution's tasks request |
Changed in mistral: | |
status: | New → In Progress |
Changed in mistral: | |
assignee: | nobody → Vasudeo Nimbekar (devnimbekar) |
To post a comment you must log in.
Reviewed: https:/ /review. opendev. org/c/openstack /mistral/ +/865985 /opendev. org/openstack/ mistral/ commit/ 87c08ff42172c9f e7735777c98d07e c32c4cd179
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 87c08ff42172c9f e7735777c98d07e c32c4cd179
Author: Vasudeo Nimbekar <email address hidden>
Date: Tue Nov 29 18:16:56 2022 +0530
Update get execution's tasks functionality
After this patch, it will validate execution-id exists in the DB while fetching tasks for the execution-id So, GET /v2/executions/ execution_ id/tasks API will return 404 for absent execution id.
Closes-Bug: #1998210 a68cd67f19d1bd9 5a4d6cebe59
Change-Id: I6ac05f4b87c470