add logger level

README.md

@@ -4,6 +4,9 @@
 
 gitlab review-bot
 
+## Preconditions
+- Gitlab version 13+
+
 ## Quick start
 
 ### Config

config/config.yaml

@@ -2,3 +2,5 @@ scm:
   host: https://gitlab.com
   token: <your-private-token>
   secret: <your-webhook-secret>
+logger:
+  level: info

global/config.go

@@ -35,6 +35,11 @@ const JWTSecret = "bot"
 type Config struct {
 	Server server.Config `json:"server"`
 	SCM    scm.Config    `json:"scm"`
+	Logger LoggerConfig  `json:"logger"`
+}
+
+type LoggerConfig struct {
+	Level string `json:"level"` // debug、info、warn、error、fatal、panic、trace
 }
 
 func Environ() *Config {

global/init.go

@@ -16,6 +16,8 @@ limitations under the License.
 package global
 
 import (
+	"fmt"
+
 	"github.com/pkgms/go/ctr"
 	"github.com/sirupsen/logrus"
 
@@ -26,6 +28,13 @@ var config *Config
 
 func InitCfg(cfg *Config) (err error) {
 	config = cfg
+
+	level, err := logrus.ParseLevel(cfg.Logger.Level)
+	if err != nil {
+		return fmt.Errorf("parse logger level failed: %v", err)
+	}
+	logrus.SetLevel(level)
+
 	logrus.SetFormatter(&logrus.TextFormatter{
 		ForceColors:            true,
 		DisableLevelTruncation: true,

handler/webhook/webhook.go

@@ -564,6 +564,10 @@ func completeAssignees(event *gitlab.MergeEvent, opt *scm.UpdatePullRequest) {
 		}
 		return ids
 	}
-	opt.AssigneeIDs = getAssigneeIDs(event)
-	opt.AssigneeID = event.Assignee.ID
+	if event.Assignees != nil {
+		opt.AssigneeIDs = getAssigneeIDs(event)
+	}
+	if event.Assignee != nil {
+		opt.AssigneeID = event.Assignee.ID
+	}
 }

pkg/scm/gitlab.go

@@ -18,6 +18,8 @@ package scm
 import (
 	"path"
 
+	"github.com/sirupsen/logrus"
+
 	"github.com/xanzy/go-gitlab"
 	"gopkg.in/yaml.v3"
 )
@@ -111,7 +113,6 @@ func (s *gitlabClient) GetPullRequest(pid string, prID int) (*PullRequest, error
 
 func (s *gitlabClient) UpdatePullRequest(pid string, prID int, data *UpdatePullRequest) error {
 	opt := &gitlab.UpdateMergeRequestOptions{
-		AssigneeIDs:  &data.AssigneeIDs,
 		AddLabels:    (*gitlab.Labels)(&data.AddLabels),
 		RemoveLabels: (*gitlab.Labels)(&data.RemoveLabels),
 	}
@@ -127,6 +128,11 @@ func (s *gitlabClient) UpdatePullRequest(pid string, prID int, data *UpdatePullR
 	if data.AssigneeID > 0 {
 		opt.AssigneeID = &data.AssigneeID
 	}
+	if len(data.AssigneeIDs) > 0 {
+		opt.AssigneeIDs = &data.AssigneeIDs
+	}
+	logrus.Debugf("UpdatePullRequest before options: %+v", opt)
+	logrus.Debugf("UpdateMergeRequest after options: %+v", opt)
 	_, _, err := s.client.MergeRequests.UpdateMergeRequest(pid, prID, opt)
 	return err
 }