diff options
Diffstat (limited to 'internal/gitaly/service/commit/languages.go')
| -rw-r--r-- | internal/gitaly/service/commit/languages.go | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/internal/gitaly/service/commit/languages.go b/internal/gitaly/service/commit/languages.go index 4363011c0..67b9d04a7 100644 --- a/internal/gitaly/service/commit/languages.go +++ b/internal/gitaly/service/commit/languages.go @@ -9,6 +9,7 @@ import ( "sort" "strings" + gitalyerrors "gitlab.com/gitlab-org/gitaly/v15/internal/errors" "gitlab.com/gitlab-org/gitaly/v15/internal/git" "gitlab.com/gitlab-org/gitaly/v15/internal/helper" "gitlab.com/gitlab-org/gitaly/v15/internal/helper/text" @@ -17,8 +18,18 @@ import ( var errAmbigRef = errors.New("ambiguous reference") -func (s *server) CommitLanguages(ctx context.Context, req *gitalypb.CommitLanguagesRequest) (*gitalypb.CommitLanguagesResponse, error) { +func (s *server) validateCommitLanguagesRequest(req *gitalypb.CommitLanguagesRequest) error { + if req.GetRepository() == nil { + return gitalyerrors.ErrEmptyRepository + } if err := git.ValidateRevisionAllowEmpty(req.Revision); err != nil { + return err + } + return nil +} + +func (s *server) CommitLanguages(ctx context.Context, req *gitalypb.CommitLanguagesRequest) (*gitalypb.CommitLanguagesResponse, error) { + if err := s.validateCommitLanguagesRequest(req); err != nil { return nil, helper.ErrInvalidArgument(err) } |