From 9eb6c4a0dfaed86713952da19a8aa5079e97addb Mon Sep 17 00:00:00 2001 From: Gusted Date: Mon, 27 Jun 2022 15:52:22 +0200 Subject: [PATCH] Add missing parameter for error in log message (#20144) - Adds a `%v` for the last parameter, `err`. Co-authored-by: John Olheiser --- routers/private/hook_pre_receive.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/routers/private/hook_pre_receive.go b/routers/private/hook_pre_receive.go index fb4c039d49..3e7d1fe9ef 100644 --- a/routers/private/hook_pre_receive.go +++ b/routers/private/hook_pre_receive.go @@ -477,7 +477,7 @@ func (ctx *preReceiveContext) loadPusherAndPermission() bool { userPerm, err := access_model.GetUserRepoPermission(ctx, ctx.Repo.Repository, user) if err != nil { - log.Error("Unable to get Repo permission of repo %s/%s of User %s", ctx.Repo.Repository.OwnerName, ctx.Repo.Repository.Name, user.Name, err) + log.Error("Unable to get Repo permission of repo %s/%s of User %s: %v", ctx.Repo.Repository.OwnerName, ctx.Repo.Repository.Name, user.Name, err) ctx.JSON(http.StatusInternalServerError, private.Response{ Err: fmt.Sprintf("Unable to get Repo permission of repo %s/%s of User %s: %v", ctx.Repo.Repository.OwnerName, ctx.Repo.Repository.Name, user.Name, err), })