MM-49703: Bump to Go 1.19 (mattermost-community#4489)

* MM-49703: Bump to Go 1.19

Also fix some gofmt issues

https://mattermost.atlassian.net/browse/MM-49703

* Bump versions in go.mod

* Update go.work as well

* Bump GolangCI to 1.50.1

* Upgraded to new builder image

.github/workflows/ci.yml

@@ -48,7 +48,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: "Test server: ${{matrix['db']}}"
       run: cd focalboard; make server-test-${{matrix['db']}}
@@ -83,7 +83,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Setup Node
       uses: actions/setup-node@v3
@@ -137,7 +137,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: "Test server (minimum): ${{matrix['db']}}"
       run: cd focalboard; make server-test-mini-${{matrix['db']}}
@@ -174,7 +174,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: "Test server (minimum): ${{matrix['db']}}"
       run: cd focalboard; make server-test-mini-${{matrix['db']}}

.github/workflows/dev-release.yml

@@ -54,7 +54,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Setup Node
       uses: actions/setup-node@v3
@@ -129,7 +129,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: List Xcode versions
       run: ls -n /Applications/ | grep Xcode*
@@ -190,7 +190,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Setup NuGet
       uses: nuget/setup-nuget@v1
@@ -258,7 +258,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Set up Node
       uses: actions/setup-node@v3

.github/workflows/lint-server.yml

@@ -30,7 +30,7 @@ jobs:
     steps:
       - uses: actions/setup-go@v3
         with:
-          go-version: 1.18.1
+          go-version: 1.19.5
       - uses: actions/checkout@v3
         with:
           path: "focalboard"
@@ -50,7 +50,7 @@ jobs:
           path: "mattermost-server"
           ref : "master"
       - name: set up golangci-lint
-        run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.46.2
+        run: curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b $(go env GOPATH)/bin v1.50.1
       - name: lint
         run: |
           cd focalboard

.github/workflows/prod-release.yml

@@ -50,7 +50,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Setup Node
       uses: actions/setup-node@v3
@@ -126,7 +126,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: List Xcode versions
       run: ls -n /Applications/ | grep Xcode*
@@ -188,7 +188,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Setup NuGet
       uses: nuget/setup-nuget@v1
@@ -257,7 +257,7 @@ jobs:
     - name: Set up Go
       uses: actions/setup-go@v3
       with:
-        go-version: 1.18.1
+        go-version: 1.19.5
 
     - name: Set up Node
       uses: actions/setup-node@v3

.gitlab-ci.yml

@@ -4,7 +4,7 @@ stages:
 
 variables:
   BUILD: "yes"
-  IMAGE_BUILDER: $CI_REGISTRY/mattermost/ci/images/builder:go-1.18.1-node-16.15.0-1
+  IMAGE_BUILDER: $CI_REGISTRY/mattermost/ci/images/builder:go-1.19.5-node-16.15.0-1
 
 include:
   - project: mattermost/ci/focalboard

build/gowork/main.go

@@ -51,7 +51,7 @@ func makeGoWork(ci bool) string {
 
 	var b strings.Builder
 
-	b.WriteString("go 1.18\n\n")
+	b.WriteString("go 1.19\n\n")
 	b.WriteString("use ./server\n")
 
 	for repo, envVarName := range repos {

linux/go.mod

@@ -1,6 +1,6 @@
 module github.com/mattermost/focalboard/linux
 
-go 1.18
+go 1.19
 
 replace github.com/mattermost/focalboard/server => ../server
 

mattermost-plugin/build/go.mod

@@ -1,6 +1,6 @@
 module github.com/mattermost/mattermost-plugin-starter-template/build
 
-go 1.18
+go 1.19
 
 require (
 	github.com/go-git/go-git/v5 v5.1.0

mattermost-plugin/go.mod

@@ -1,6 +1,6 @@
 module github.com/mattermost/focalboard/mattermost-plugin
 
-go 1.18
+go 1.19
 
 require (
 	github.com/golang/mock v1.6.0

server/.golangci.yml

@@ -27,10 +27,7 @@ linters:
   enable:
     - gofmt
     - goimports
-    - deadcode
     - ineffassign
-    - structcheck
-    - varcheck
     - unparam
     - errcheck
     - govet

server/api/api.go

@@ -220,7 +220,7 @@ func stringResponse(w http.ResponseWriter, message string) {
 	_, _ = fmt.Fprint(w, message)
 }
 
-func jsonStringResponse(w http.ResponseWriter, code int, message string) {
+func jsonStringResponse(w http.ResponseWriter, code int, message string) { //nolint:unparam
 	setResponseHeader(w, "Content-Type", "application/json")
 	w.WriteHeader(code)
 	fmt.Fprint(w, message)
@@ -232,7 +232,7 @@ func jsonBytesResponse(w http.ResponseWriter, code int, json []byte) {
 	_, _ = w.Write(json)
 }
 
-func setResponseHeader(w http.ResponseWriter, key string, value string) {
+func setResponseHeader(w http.ResponseWriter, key string, value string) { //nolint:unparam
 	header := w.Header()
 	if header == nil {
 		return

server/api/audit.go

@@ -8,7 +8,7 @@ import (
 )
 
 // makeAuditRecord creates an audit record pre-populated with data from the request.
-func (a *API) makeAuditRecord(r *http.Request, event string, initialStatus string) *audit.Record {
+func (a *API) makeAuditRecord(r *http.Request, event string, initialStatus string) *audit.Record { //nolint:unparam
 	ctx := r.Context()
 	var sessionID string
 	var userID string

server/api/auth.go

@@ -383,12 +383,13 @@ func (a *API) attachSession(handler func(w http.ResponseWriter, r *http.Request)
 
 		authService := session.AuthService
 		if authService != a.authService {
-			a.logger.Error(`Session authService mismatch`,
+			msg := `Session authService mismatch`
+			a.logger.Error(msg,
 				mlog.String("sessionID", session.ID),
 				mlog.String("want", a.authService),
 				mlog.String("got", authService),
 			)
-			a.errorResponse(w, r, model.NewErrUnauthorized(err.Error()))
+			a.errorResponse(w, r, model.NewErrUnauthorized(msg))
 			return
 		}
 

server/api/members.go

@@ -146,7 +146,7 @@ func (a *API) handleAddMember(w http.ResponseWriter, r *http.Request) {
 	}
 
 	if reqBoardMember.UserID == "" {
-		a.errorResponse(w, r, model.NewErrBadRequest(err.Error()))
+		a.errorResponse(w, r, model.NewErrBadRequest("empty userID"))
 		return
 	}
 

server/app/insights.go

@@ -19,7 +19,7 @@ func (a *App) GetTeamBoardsInsights(userID string, teamID string, opts *mmModel.
 	if err != nil {
 		return nil, err
 	}
-	return a.store.GetTeamBoardsInsights(teamID, userID, opts.StartUnixMilli, opts.Page*opts.PerPage, opts.PerPage, boardIDs)
+	return a.store.GetTeamBoardsInsights(teamID, opts.StartUnixMilli, opts.Page*opts.PerPage, opts.PerPage, boardIDs)
 }
 
 func (a *App) GetUserBoardsInsights(userID string, teamID string, opts *mmModel.InsightsOpts) (*model.BoardInsightsList, error) {

server/app/insights_test.go

@@ -51,7 +51,7 @@ func TestGetTeamAndUserBoardsInsights(t *testing.T) {
 		th.Store.EXPECT().GetUserByID("user-id").Return(fakeUser, nil).AnyTimes()
 		th.Store.EXPECT().GetBoardsForUserAndTeam("user-id", "team-id", true).Return(mockInsightsBoards, nil).AnyTimes()
 		th.Store.EXPECT().
-			GetTeamBoardsInsights("team-id", "user-id", int64(0), 0, 10, []string{"mock-user-workspace-id"}).
+			GetTeamBoardsInsights("team-id", int64(0), 0, 10, []string{"mock-user-workspace-id"}).
 			Return(mockTeamInsightsList, nil)
 		results, err := th.App.GetTeamBoardsInsights("user-id", "team-id", &mmModel.InsightsOpts{StartUnixMilli: 0, Page: 0, PerPage: 10})
 		require.NoError(t, err)
@@ -74,7 +74,7 @@ func TestGetTeamAndUserBoardsInsights(t *testing.T) {
 		th.Store.EXPECT().GetUserByID("user-id").Return(fakeUser, nil).AnyTimes()
 		th.Store.EXPECT().GetBoardsForUserAndTeam("user-id", "team-id", true).Return(mockInsightsBoards, nil).AnyTimes()
 		th.Store.EXPECT().
-			GetTeamBoardsInsights("team-id", "user-id", int64(0), 0, 10, []string{"mock-user-workspace-id"}).
+			GetTeamBoardsInsights("team-id", int64(0), 0, 10, []string{"mock-user-workspace-id"}).
 			Return(nil, insightError{"board-insight-error"})
 		_, err := th.App.GetTeamBoardsInsights("user-id", "team-id", &mmModel.InsightsOpts{StartUnixMilli: 0, Page: 0, PerPage: 10})
 		require.Error(t, err)

server/assets/assets.go

@@ -7,5 +7,6 @@ import (
 // DefaultTemplatesArchive is an embedded archive file containing the default
 // templates to be imported to team 0.
 // This archive is generated with `make templates-archive`
+//
 //go:embed templates.boardarchive
 var DefaultTemplatesArchive []byte

server/go.mod

@@ -1,6 +1,6 @@
 module github.com/mattermost/focalboard/server
 
-go 1.18
+go 1.19
 
 require (
 	github.com/Masterminds/squirrel v1.5.3

server/go.tools.mod

@@ -1,6 +1,6 @@
 module github.com/mattermost/focalboard/server
 
-go 1.18
+go 1.19
 
 require github.com/golang/mock v1.6.0
 

server/integrationtests/cards_test.go

@@ -269,9 +269,7 @@ func TestGetCard(t *testing.T) {
 	})
 }
 
-//
 // Helpers.
-//
 func reverse(src []string) []string {
 	out := make([]string, 0, len(src))
 	for i := len(src) - 1; i >= 0; i-- {

server/main/main.go

@@ -122,7 +122,7 @@ func main() {
 	if pDBConfig != nil && len(*pDBConfig) > 0 {
 		config.DBConfigString = *pDBConfig
 		// Don't echo, as the confix string may contain passwords
-		logger.Info("DBConfigString overriden from commandline")
+		logger.Info("DBConfigString overridden from commandline")
 	}
 
 	if pPort != nil && *pPort > 0 && *pPort != config.Port {
@@ -166,6 +166,7 @@ func main() {
 }
 
 // StartServer starts the server
+//
 //export StartServer
 func StartServer(webPath *C.char, filesPath *C.char, port int, singleUserToken, dbConfigString, configFilePath *C.char) {
 	startServer(
@@ -179,6 +180,7 @@ func StartServer(webPath *C.char, filesPath *C.char, port int, singleUserToken,
 }
 
 // StopServer stops the server
+//
 //export StopServer
 func StopServer() {
 	stopServer()

server/server/server.go

@@ -377,7 +377,7 @@ func (s *Server) UpdateAppConfig() {
 // Local server
 
 func (s *Server) startLocalModeServer() error {
-	s.localModeServer = &http.Server{
+	s.localModeServer = &http.Server{ //nolint:gosec
 		Handler:     s.localRouter,
 		ConnContext: api.SetContextConn,
 	}

server/services/audit/audit.go

@@ -46,12 +46,14 @@ func NewAudit(options ...mlog.Option) (*Audit, error) {
 
 // Configure provides a new configuration for this audit service.
 // Zero or more sources of config can be provided:
-//   cfgFile    - path to file containing JSON
-//   cfgEscaped - JSON string probably from ENV var
+//
+//	cfgFile    - path to file containing JSON
+//	cfgEscaped - JSON string probably from ENV var
 //
 // For each case JSON containing log targets is provided. Target name collisions are resolved
 // using the following precedence:
-//     cfgFile > cfgEscaped
+//
+//	cfgFile > cfgEscaped
 func (a *Audit) Configure(cfgFile string, cfgEscaped string) error {
 	return a.auditLogger.Configure(cfgFile, cfgEscaped, nil)
 }

server/services/metrics/service.go

@@ -17,7 +17,7 @@ type Service struct {
 // NewMetricsServer factory method to create a new prometheus server.
 func NewMetricsServer(address string, metricsService *Metrics, logger mlog.LoggerIFace) *Service {
 	return &Service{
-		&http.Server{
+		&http.Server{ //nolint:gosec
 			Addr: address,
 			Handler: promhttp.HandlerFor(metricsService.registry, promhttp.HandlerOpts{
 				ErrorLog: logger.StdLogger(mlog.LvlError),

server/services/store/mattermostauthlayer/mattermostauthlayer.go

@@ -599,7 +599,7 @@ func (s *MattermostAuthLayer) GetLicense() *mmModel.License {
 	return s.servicesAPI.GetLicense()
 }
 
-func boardFields(prefix string) []string {
+func boardFields(prefix string) []string { //nolint:unparam
 	fields := []string{
 		"id",
 		"team_id",

server/services/store/sqlstore/board_insights.go

@@ -14,7 +14,7 @@ import (
 	"github.com/mattermost/mattermost-server/v6/shared/mlog"
 )
 
-func (s *SQLStore) getTeamBoardsInsights(db sq.BaseRunner, teamID string, userID string, since int64, offset int, limit int, boardIDs []string) (*model.BoardInsightsList, error) {
+func (s *SQLStore) getTeamBoardsInsights(db sq.BaseRunner, teamID string, since int64, offset int, limit int, boardIDs []string) (*model.BoardInsightsList, error) {
 	boardsHistoryQuery := s.getQueryBuilder(db).
 		Select("boards.id, boards.icon, boards.title, count(boards_history.id) as count, boards_history.modified_by, boards.created_by").
 		From(s.tablePrefix + "boards_history as boards_history").