diff options
author | David Kim <dkim@gitlab.com> | 2020-04-07 15:12:10 +0930 |
---|---|---|
committer | David Kim <dkim@gitlab.com> | 2020-04-07 15:16:32 +0930 |
commit | a9d9243401d887083477079192eb30452f7ccfc3 (patch) | |
tree | 3f3f6d1f8cf13978e611a3c683a904727f2ba39f /internal/command/uploadpack | |
parent | 12c6711f321dd4b0acf507041cf05fbaa623e29e (diff) | |
download | gitlab-shell-a9d9243401d887083477079192eb30452f7ccfc3.tar.gz |
Move logging to handler instead
Diffstat (limited to 'internal/command/uploadpack')
-rw-r--r-- | internal/command/uploadpack/gitalycall.go | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/internal/command/uploadpack/gitalycall.go b/internal/command/uploadpack/gitalycall.go index 8ab163d..d33fac8 100644 --- a/internal/command/uploadpack/gitalycall.go +++ b/internal/command/uploadpack/gitalycall.go @@ -9,7 +9,6 @@ import ( "gitlab.com/gitlab-org/gitaly/client" pb "gitlab.com/gitlab-org/gitaly/proto/go/gitalypb" "gitlab.com/gitlab-org/gitlab-shell/internal/command/commandargs" - "gitlab.com/gitlab-org/gitlab-shell/internal/command/shared/commandlogger" "gitlab.com/gitlab-org/gitlab-shell/internal/gitlabnet/accessverifier" "gitlab.com/gitlab-org/gitlab-shell/internal/handler" ) @@ -29,12 +28,12 @@ func (c *Command) performGitalyCall(response *accessverifier.Response) error { GitConfigOptions: response.GitConfigOptions, } - commandlogger.Log("git-upload-pack", request.Repository, response, request.GitProtocol) - return gc.RunGitalyCommand(func(ctx context.Context, conn *grpc.ClientConn) (int32, error) { ctx, cancel := context.WithCancel(ctx) defer cancel() + gc.LogExecution("git-upload-pack", request.Repository, response, request.GitProtocol) + rw := c.ReadWriter return client.UploadPack(ctx, conn, rw.In, rw.Out, rw.ErrOut, request) }) |