all: fix linting issues

Change-Id: Idfc93948e59a181321d79b365e638d63e256a16f

certificate/certificatepb/gen.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

cmd/storagenode-updater/loop_windows.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2020 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build windows && !unittest
 // +build windows,!unittest
 
 package main

cmd/storagenode-updater/main_windows.go

@@ -7,6 +7,7 @@
 //
 // sc.exe create storagenode-updater binpath= "C:\Users\MyUser\storagenode-updater.exe run ..."
 
+//go:build windows
 // +build windows
 
 package main

cmd/storagenode-updater/restart_linux.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2020 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build linux && service
 // +build linux,service
 
 package main

cmd/storagenode-updater/restart_windows.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2020 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build windows && service
 // +build windows,service
 
 package main

cmd/storagenode/service_windows.go

@@ -61,7 +61,7 @@ func (m *service) Execute(args []string, r <-chan svc.ChangeRequest, changes cha
 		process.Exec(rootCmd)
 		return nil
 	})
-	defer group.Wait()
+	defer func() { _ = group.Wait() }()
 
 	changes <- svc.Status{State: svc.Running, Accepts: cmdsAccepted}
 

private/crashreportpb/gen.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

private/cui/example.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

private/multinodepb/gen.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

satellite/admin/oauthclients_testplanet_test.go

@@ -75,7 +75,9 @@ func TestAdminOAuthAPI(t *testing.T) {
 
 			body := ""
 			if testCase.request != nil {
-				if data, _ := json.Marshal(testCase.request); len(data) > 0 {
+				data, err := json.Marshal(testCase.request)
+				require.NoError(t, err)
+				if len(data) > 0 {
 					body = string(data)
 				}
 			}

satellite/console/wasm/main.go

@@ -1,4 +1,6 @@
+//go:build js && wasm
 // +build js,wasm
+
 // Copyright (C) 2020 Storj Labs, Inc.
 // See LICENSE for copying information.
 

satellite/internalpb/gen.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

satellite/oidc/oauth_stores.go

@@ -12,12 +12,15 @@ import (
 	"storj.io/common/uuid"
 )
 
-// clientStore provides a simple adapter for the oauth implementation.
-type clientStore struct {
+// ClientStore provides a simple adapter for the oauth implementation.
+type ClientStore struct {
 	clients OAuthClients
 }
 
-func (c *clientStore) GetByID(ctx context.Context, id string) (oauth2.ClientInfo, error) {
+var _ oauth2.ClientStore = (*ClientStore)(nil)
+
+// GetByID returns client information by id.
+func (c *ClientStore) GetByID(ctx context.Context, id string) (oauth2.ClientInfo, error) {
 	uid, err := uuid.FromString(id)
 	if err != nil {
 		return nil, err
@@ -26,13 +29,16 @@ func (c *clientStore) GetByID(ctx context.Context, id string) (oauth2.ClientInfo
 	return c.clients.Get(ctx, uid)
 }
 
-// tokenStore provides a simple adapter for the oauth implementation.
-type tokenStore struct {
+// TokenStore provides a simple adapter for the oauth implementation.
+type TokenStore struct {
 	codes  OAuthCodes
 	tokens OAuthTokens
 }
 
-func (t *tokenStore) Create(ctx context.Context, info oauth2.TokenInfo) (err error) {
+var _ oauth2.TokenStore = (*TokenStore)(nil)
+
+// Create creates a new token with the given info.
+func (t *TokenStore) Create(ctx context.Context, info oauth2.TokenInfo) (err error) {
 	var code OAuthCode
 	var access, refresh OAuthToken
 
@@ -108,19 +114,23 @@ func (t *tokenStore) Create(ctx context.Context, info oauth2.TokenInfo) (err err
 	return nil
 }
 
-func (t *tokenStore) RemoveByCode(ctx context.Context, code string) error {
+// RemoveByCode deletes token by authorization code.
+func (t *TokenStore) RemoveByCode(ctx context.Context, code string) error {
 	return t.codes.Claim(ctx, code)
 }
 
-func (t *tokenStore) RemoveByAccess(ctx context.Context, access string) error {
+// RemoveByAccess deletes token by access token.
+func (t *TokenStore) RemoveByAccess(ctx context.Context, access string) error {
 	return nil // unsupported by current configuration
 }
 
-func (t *tokenStore) RemoveByRefresh(ctx context.Context, refresh string) error {
+// RemoveByRefresh deletes token by refresh token.
+func (t *TokenStore) RemoveByRefresh(ctx context.Context, refresh string) error {
 	return nil // unsupported by current configuration
 }
 
-func (t *tokenStore) GetByCode(ctx context.Context, code string) (oauth2.TokenInfo, error) {
+// GetByCode uses authorization code to find token information.
+func (t *TokenStore) GetByCode(ctx context.Context, code string) (oauth2.TokenInfo, error) {
 	oauthCode, err := t.codes.Get(ctx, code)
 	if err != nil {
 		return nil, err
@@ -129,7 +139,8 @@ func (t *tokenStore) GetByCode(ctx context.Context, code string) (oauth2.TokenIn
 	return &record{code: oauthCode}, nil
 }
 
-func (t *tokenStore) GetByAccess(ctx context.Context, access string) (oauth2.TokenInfo, error) {
+// GetByAccess uses access token to find token information.
+func (t *TokenStore) GetByAccess(ctx context.Context, access string) (oauth2.TokenInfo, error) {
 	oauthToken, err := t.tokens.Get(ctx, KindAccessToken, access)
 	if err != nil {
 		return nil, err
@@ -138,7 +149,8 @@ func (t *tokenStore) GetByAccess(ctx context.Context, access string) (oauth2.Tok
 	return &record{access: oauthToken}, nil
 }
 
-func (t *tokenStore) GetByRefresh(ctx context.Context, refresh string) (oauth2.TokenInfo, error) {
+// GetByRefresh uses refresh token to find token information.
+func (t *TokenStore) GetByRefresh(ctx context.Context, refresh string) (oauth2.TokenInfo, error) {
 	oauthToken, err := t.tokens.Get(ctx, KindRefreshToken, refresh)
 	if err != nil {
 		return nil, err

satellite/oidc/service.go

@@ -18,15 +18,15 @@ type Service struct {
 }
 
 // ClientStore returns a store used to lookup oauth clients from the consent flow.
-func (s *Service) ClientStore() *clientStore {
-	return &clientStore{
+func (s *Service) ClientStore() *ClientStore {
+	return &ClientStore{
 		clients: s.store.OAuthClients(),
 	}
 }
 
 // TokenStore returns a store used to manage access tokens during the consent flow.
-func (s *Service) TokenStore() *tokenStore {
-	return &tokenStore{
+func (s *Service) TokenStore() *TokenStore {
+	return &TokenStore{
 		codes:  s.store.OAuthCodes(),
 		tokens: s.store.OAuthTokens(),
 	}

satellite/satellitedb/migrate_gen.go

@@ -1,7 +1,8 @@
 // Copyright (C) 2021 Storj Labs, Inc.
 // See LICENSE for copying information.
 
-//+build ignore
+//go:build ignore
+// +build ignore
 
 package main
 

satellite/satellitedb/migrate_test.go

@@ -84,7 +84,7 @@ func loadSnapshots(ctx context.Context, connstr, dbxscript string) (*dbschema.Sn
 			if err != nil {
 				var pgErr *pgconn.PgError
 				if errors.As(err, &pgErr) {
-					return fmt.Errorf("Version %d error: %v\nDetail: %s\nHint: %s", version, pgErr, pgErr.Detail, pgErr.Hint)
+					return fmt.Errorf("Version %d error: %w\nDetail: %s\nHint: %s", version, pgErr, pgErr.Detail, pgErr.Hint)
 				}
 				return fmt.Errorf("Version %d error: %w", version, err)
 			}

scripts/pg-to-crdb.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

scripts/test-network-stalls.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 // Tests whether the uplink tool correctly times out when one of the storage nodes it's talking to

scripts/update-access.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2020 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

storage/filestore/dir_windows.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build windows
 // +build windows
 
 package filestore

storagenode/internalpb/gen.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2019 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main

storagenode/storagenodedb/schemagen.go

@@ -1,6 +1,7 @@
 // Copyright (C) 2020 Storj Labs, Inc.
 // See LICENSE for copying information.
 
+//go:build ignore
 // +build ignore
 
 package main