From 657601b858a8a094562db962c4cc11f129561ba7 Mon Sep 17 00:00:00 2001 From: Drew DeVault Date: Fri, 21 Aug 2020 09:44:09 -0400 Subject: [PATCH] update-hook: correct error in manifest limit --- gitsrht-update-hook/submitter.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/gitsrht-update-hook/submitter.go b/gitsrht-update-hook/submitter.go index fb71c4b..6eb085b 100644 --- a/gitsrht-update-hook/submitter.go +++ b/gitsrht-update-hook/submitter.go @@ -249,13 +249,13 @@ func SubmitBuild(submitter BuildSubmitter) ([]BuildSubmission, error) { return nil, nil } - if len(manifests) > 4 { - manifests = manifests[:4] - log.Println("Notice: refusing to submit >4 builds") - } - var results []BuildSubmission for name, contents := range manifests { + if len(results) > 4 { + log.Println("Notice: refusing to submit >4 builds") + break + } + manifest, err := ManifestFromYAML(contents) if err != nil { return nil, errors.Wrap(err, name) -- 2.38.4