Enable revive linter in golangci-lint and solve issues (STACKED ON #3487)

.golangci.yaml

@@ -41,7 +41,6 @@ linters:
     - containedctx      # TODO: consider enabling the 'containedctx' linter to detect struct contained context.Context field.
     - whitespace        # TODO: consider enabling the 'whitespace' linter to detect leading and trailing whitespaces.
     - unparam           # TODO: consider enabling the 'unparam' linter to report unused function parameters.
-    - revive            # TODO: consider enabling the 'revive' linter to configure custom rules and define a strict preset for enhancing development & code review processes.
     - nakedret          # TODO: consider enabling the 'nakedret' linter to find naked returns in functions greater than a specified function length.
     - dupword           # TODO: consider enabling the 'dupword' linter to check for duplicate words in the source code.
     - makezero          # TODO: consider enabling the 'makezero' linter to find slice declarations with non-zero initial length.

changelog/unreleased/enhancement-revive.md

@@ -0,0 +1,3 @@
+Enhancement: Enable revive linter in golangci-lint and solve issues
+
+https://github.com/cs3org/reva/pull/3465

internal/http/services/ocmd/shares.go

@@ -141,7 +141,7 @@ func (h *sharesHandler) createShare(w http.ResponseWriter, r *http.Request) {
 		return
 	}
 
-	var shareWithParts []string = strings.Split(shareWith, "@")
+	var shareWithParts = strings.Split(shareWith, "@")
 	userRes, err := gatewayClient.GetUser(ctx, &userpb.GetUserRequest{
 		UserId: &userpb.UserId{OpaqueId: shareWithParts[0]}, SkipFetchingUserGroups: true,
 	})

pkg/app/provider/wopi/wopi.go

@@ -161,7 +161,7 @@ func (p *wopiProvider) GetAppURL(ctx context.Context, resource *provider.Resourc
 			q.Add("appviewurl", viewAppURL)
 		}
 	}
-	var access string = "edit"
+	var access = "edit"
 	if resource.GetSize() == 0 {
 		if _, ok := p.appURLs["editnew"]; ok {
 			access = "editnew"

pkg/ocm/invite/manager/json/json.go

@@ -239,7 +239,7 @@ func (m *manager) ForwardInvite(ctx context.Context, invite *invitepb.InviteToke
 		if e != nil {
 			return errors.Wrap(e, "json: error reading request body")
 		}
-		return errors.Wrap(errors.New(fmt.Sprintf("%s: %s", resp.Status, string(respBody))), "json: error sending accept post request")
+		return errors.Wrap(fmt.Errorf("%s: %s", resp.Status, string(respBody)), "json: error sending accept post request")
 	}
 
 	return nil

pkg/sysinfo/sysinfo.go

@@ -34,7 +34,7 @@ type SystemInformation struct {
 
 var (
 	// SysInfo provides global system information.
-	SysInfo *SystemInformation = &SystemInformation{}
+	SysInfo = &SystemInformation{}
 )
 
 // ToJSON converts the system information to JSON.