blender/flamenco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Manager: reject removal of empty list of blocklist entries

Browse Source 
A request to remove an empty list of blocklist entries now results in a
400 Bad Request.
This commit is contained in:
Sybren A. Stüvel 2022-08-01 18:55:33 +02:00
parent 3b978ceda0
commit 11e5363d24
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
internal/manager/api_impl/jobs.go
View File

@ -376,6 +376,9 @@ func (f *Flamenco) RemoveJobBlocklist(e echo.Context, jobID string) error {
logger.Warn().Err(err).Msg("bad request received") logger.Warn().Err(err).Msg("bad request received")
return sendAPIError(e, http.StatusBadRequest, "invalid format") return sendAPIError(e, http.StatusBadRequest, "invalid format")
} }
if len(entriesToRemove) == 0 {
return sendAPIError(e, http.StatusBadRequest, "empty list of blocklist entries given")
}
var lastErr error var lastErr error
for _, entry := range entriesToRemove { for _, entry := range entriesToRemove {