From 7cccada51eb925e7acfc90aae26661afd847d8f4 Mon Sep 17 00:00:00 2001 From: Daniel Fonseca de Lira Date: Tue, 24 Sep 2019 03:55:07 -0300 Subject: [PATCH] Fix assets key on release webhook (#8253) * Fix assets key on release webhook * Fix assets key on release webhook * Add message to log on AddReleaseAttachments failure --- services/release/release.go | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/services/release/release.go b/services/release/release.go index 4451633798..246a784efe 100644 --- a/services/release/release.go +++ b/services/release/release.go @@ -109,12 +109,14 @@ func UpdateRelease(doer *models.User, gitRepo *git.Repository, rel *models.Relea return err } + if err = models.AddReleaseAttachments(rel.ID, attachmentUUIDs); err != nil { + log.Error("AddReleaseAttachments: %v", err) + } + if err = rel.LoadAttributes(); err != nil { return err } - err = models.AddReleaseAttachments(rel.ID, attachmentUUIDs) - // even if attachments added failed, hooks will be still triggered mode, _ := models.AccessLevel(doer, rel.Repo) if err1 := models.PrepareWebhooks(rel.Repo, models.HookEventRelease, &api.ReleasePayload{