From 2b53a932bfdede083db45017be73f8369b28fc24 Mon Sep 17 00:00:00 2001
From: Vasco Guita <vasco.guita@tecnico.ulisboa.pt>
Date: Wed, 23 Nov 2022 14:17:20 +0100
Subject: [PATCH] Enable the goimports and usestdlibvars linters and solve
 issues

---
 .golangci.yaml                                |  2 --
 changelog/unreleased/enhancement-goimports.md |  5 +++++
 cmd/reva/download.go                          |  2 +-
 cmd/reva/upload.go                            |  2 +-
 internal/http/interceptors/auth/auth.go       |  2 +-
 internal/http/interceptors/cors/cors.go       | 12 +++++++-----
 .../providerauthorizer/providerauthorizer.go  |  2 +-
 .../http/services/datagateway/datagateway.go  | 16 ++++++++--------
 internal/http/services/owncloud/ocdav/copy.go |  4 ++--
 internal/http/services/owncloud/ocdav/tpc.go  |  8 ++++----
 internal/http/services/owncloud/ocs/cache.go  |  2 +-
 pkg/app/provider/demo/demo.go                 |  3 ++-
 pkg/app/provider/wopi/wopi.go                 | 10 +++++-----
 pkg/cbox/utils/tokenmanagement.go             |  4 ++--
 pkg/datatx/manager/rclone/rclone.go           |  8 ++++----
 pkg/eosclient/eosgrpc/eoshttp.go              | 19 +++++++++----------
 pkg/metrics/driver/xcloud/xcloud.go           |  2 +-
 pkg/ocm/invite/manager/json/json.go           |  2 +-
 pkg/ocm/invite/manager/memory/memory.go       |  2 +-
 pkg/ocm/provider/authorizer/mentix/mentix.go  |  2 +-
 pkg/ocm/share/sender/sender.go                |  2 +-
 pkg/rhttp/datatx/manager/simple/simple.go     |  4 ++--
 pkg/rhttp/datatx/manager/spaces/spaces.go     |  4 ++--
 pkg/rhttp/datatx/manager/tus/tus.go           |  8 ++++----
 pkg/rhttp/datatx/utils/download/download.go   |  2 +-
 pkg/sdk/action/download.go                    |  3 ++-
 pkg/sdk/action/upload.go                      |  3 ++-
 pkg/sdk/common/net/tus.go                     |  2 +-
 pkg/siteacc/manager/gocdb/account.go          |  2 +-
 pkg/storage/fs/cback/cback.go                 |  2 +-
 pkg/storage/fs/nextcloud/nextcloud.go         |  4 ++--
 31 files changed, 76 insertions(+), 69 deletions(-)
 create mode 100644 changelog/unreleased/enhancement-goimports.md

diff --git a/.golangci.yaml b/.golangci.yaml
index 3c0ae8f5b2..6b077f56f2 100644
--- a/.golangci.yaml
+++ b/.golangci.yaml
@@ -8,12 +8,10 @@ linters:
     - wrapcheck         # TODO: consider enabling the 'wrapcheck' linter to check that errors from external packages are wrapped during return to help identify the error source during debugging.
     - cyclop            # TODO: consider enabling the 'cyclop' linter to calculate the cyclomatic complexities of functions/packages.
     - varnamelen        # TODO: consider enabling the 'varnamelen' linter to check that the length of a variable's name matches its usage scope.
-    - goimports         # TODO: consider enabling the 'goimports' linter to fix imports and format the code in the same style as gofmt.
     - testpackage       # TODO: consider enabling the 'testpackage' linter to make sure that separate _test packages are used.
     - gosec             # TODO: consider enabling the 'gosec' linter to inspect source code for security problems.
     - tagliatelle       # TODO: consider enabling the 'tagliatelle' linter to check the struct tags.
     - stylecheck        # TODO: consider enabling the 'stylecheck' linter to enforce style rules.
-    - usestdlibvars     # TODO: consider enabling the 'usestdlibvars' linter to detect the possibility to use variables/constants from the Go standard library.
     - thelper           # TODO: consider enabling the 'thelper' linter to detect golang test helpers without t.Helper() call and check the consistency of test helpers.
     - predeclared       # TODO: consider enabling the 'predeclared' linter to find code that shadows one of Go's predeclared identifiers.
     - paralleltest      # TODO: consider enabling the 'paralleltest' linter to detect missing usage of t.Parallel() method in Go test.
diff --git a/changelog/unreleased/enhancement-goimports.md b/changelog/unreleased/enhancement-goimports.md
new file mode 100644
index 0000000000..002a524f77
--- /dev/null
+++ b/changelog/unreleased/enhancement-goimports.md
@@ -0,0 +1,5 @@
+Enhancement: Enable goimports and usestdlibvars in golangci-lint
+
+We've enabled the goimports and usestdlibvars linters in golangci-lint and solved the related issues.
+
+https://github.com/cs3org/reva/pull/3471
\ No newline at end of file
diff --git a/cmd/reva/download.go b/cmd/reva/download.go
index 82da692842..716101a27f 100644
--- a/cmd/reva/download.go
+++ b/cmd/reva/download.go
@@ -95,7 +95,7 @@ func downloadCommand() *command {
 
 			dataServerURL := p.DownloadEndpoint
 			// TODO(labkode): do a protocol switch
-			httpReq, err := rhttp.NewRequest(ctx, "GET", dataServerURL, nil)
+			httpReq, err := rhttp.NewRequest(ctx, http.MethodGet, dataServerURL, nil)
 			if err != nil {
 				return err
 			}
diff --git a/cmd/reva/upload.go b/cmd/reva/upload.go
index 9dd54df050..8fb567e538 100644
--- a/cmd/reva/upload.go
+++ b/cmd/reva/upload.go
@@ -148,7 +148,7 @@ func uploadCommand() *command {
 		dataServerURL := p.UploadEndpoint
 
 		if *protocolFlag == "simple" {
-			httpReq, err := rhttp.NewRequest(ctx, "PUT", dataServerURL, fd)
+			httpReq, err := rhttp.NewRequest(ctx, http.MethodPut, dataServerURL, fd)
 			if err != nil {
 				return err
 			}
diff --git a/internal/http/interceptors/auth/auth.go b/internal/http/interceptors/auth/auth.go
index 3348e07947..cca7cc4e17 100644
--- a/internal/http/interceptors/auth/auth.go
+++ b/internal/http/interceptors/auth/auth.go
@@ -158,7 +158,7 @@ func New(m map[string]interface{}, unprotected []string) (global.Middleware, err
 			// OPTION requests need to pass for preflight requests
 			// TODO(labkode): this will break options for auth protected routes.
 			// Maybe running the CORS middleware before auth kicks in is enough.
-			if r.Method == "OPTIONS" {
+			if r.Method == http.MethodOptions {
 				h.ServeHTTP(w, r)
 				return
 			}
diff --git a/internal/http/interceptors/cors/cors.go b/internal/http/interceptors/cors/cors.go
index 1d7080ac09..0d978fb929 100644
--- a/internal/http/interceptors/cors/cors.go
+++ b/internal/http/interceptors/cors/cors.go
@@ -19,6 +19,8 @@
 package cors
 
 import (
+	"net/http"
+
 	"github.com/cs3org/reva/pkg/rhttp/global"
 	"github.com/mitchellh/mapstructure"
 	"github.com/rs/cors"
@@ -62,11 +64,11 @@ func New(m map[string]interface{}) (global.Middleware, int, error) {
 
 	if len(conf.AllowedMethods) == 0 {
 		conf.AllowedMethods = []string{
-			"OPTIONS",
-			"HEAD",
-			"GET",
-			"PUT",
-			"POST",
+			http.MethodOptions,
+			http.MethodHead,
+			http.MethodGet,
+			http.MethodPut,
+			http.MethodPost,
 			"DELETE",
 			"MKCOL",
 			"PROPFIND",
diff --git a/internal/http/interceptors/providerauthorizer/providerauthorizer.go b/internal/http/interceptors/providerauthorizer/providerauthorizer.go
index 450614dcc8..76ac951f80 100644
--- a/internal/http/interceptors/providerauthorizer/providerauthorizer.go
+++ b/internal/http/interceptors/providerauthorizer/providerauthorizer.go
@@ -79,7 +79,7 @@ func New(m map[string]interface{}, unprotected []string, ocmPrefix string) (glob
 			log := appctx.GetLogger(ctx)
 			head, _ := router.ShiftPath(r.URL.Path)
 
-			if r.Method == "OPTIONS" || head != ocmPrefix || utils.Skip(r.URL.Path, unprotected) {
+			if r.Method == http.MethodOptions || head != ocmPrefix || utils.Skip(r.URL.Path, unprotected) {
 				log.Info().Msg("skipping provider authorizer check for: " + r.URL.Path)
 				h.ServeHTTP(w, r)
 				return
diff --git a/internal/http/services/datagateway/datagateway.go b/internal/http/services/datagateway/datagateway.go
index dba255f6be..65d201e801 100644
--- a/internal/http/services/datagateway/datagateway.go
+++ b/internal/http/services/datagateway/datagateway.go
@@ -117,17 +117,17 @@ func (s *svc) Unprotected() []string {
 func (s *svc) setHandler() {
 	s.handler = http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 		switch r.Method {
-		case "HEAD":
+		case http.MethodHead:
 			addCorsHeader(w)
 			s.doHead(w, r)
 			return
-		case "GET":
+		case http.MethodGet:
 			s.doGet(w, r)
 			return
-		case "PUT":
+		case http.MethodPut:
 			s.doPut(w, r)
 			return
-		case "PATCH":
+		case http.MethodPatch:
 			s.doPatch(w, r)
 			return
 		default:
@@ -183,7 +183,7 @@ func (s *svc) doHead(w http.ResponseWriter, r *http.Request) {
 	log.Debug().Str("target", claims.Target).Msg("sending request to internal data server")
 
 	httpClient := s.client
-	httpReq, err := rhttp.NewRequest(ctx, "HEAD", claims.Target, nil)
+	httpReq, err := rhttp.NewRequest(ctx, http.MethodHead, claims.Target, nil)
 	if err != nil {
 		log.Error().Err(err).Msg("wrong request")
 		w.WriteHeader(http.StatusInternalServerError)
@@ -229,7 +229,7 @@ func (s *svc) doGet(w http.ResponseWriter, r *http.Request) {
 	log.Debug().Str("target", claims.Target).Msg("sending request to internal data server")
 
 	httpClient := s.client
-	httpReq, err := rhttp.NewRequest(ctx, "GET", claims.Target, nil)
+	httpReq, err := rhttp.NewRequest(ctx, http.MethodGet, claims.Target, nil)
 	if err != nil {
 		log.Error().Err(err).Msg("wrong request")
 		w.WriteHeader(http.StatusInternalServerError)
@@ -300,7 +300,7 @@ func (s *svc) doPut(w http.ResponseWriter, r *http.Request) {
 	log.Debug().Str("target", claims.Target).Msg("sending request to internal data server")
 
 	httpClient := s.client
-	httpReq, err := rhttp.NewRequest(ctx, "PUT", target, r.Body)
+	httpReq, err := rhttp.NewRequest(ctx, http.MethodPut, target, r.Body)
 	if err != nil {
 		log.Err(err).Msg("wrong request")
 		w.WriteHeader(http.StatusInternalServerError)
@@ -359,7 +359,7 @@ func (s *svc) doPatch(w http.ResponseWriter, r *http.Request) {
 	log.Debug().Str("target", claims.Target).Msg("sending request to internal data server")
 
 	httpClient := s.client
-	httpReq, err := rhttp.NewRequest(ctx, "PATCH", target, r.Body)
+	httpReq, err := rhttp.NewRequest(ctx, http.MethodPatch, target, r.Body)
 	if err != nil {
 		log.Err(err).Msg("wrong request")
 		w.WriteHeader(http.StatusInternalServerError)
diff --git a/internal/http/services/owncloud/ocdav/copy.go b/internal/http/services/owncloud/ocdav/copy.go
index 55a6ea6fc0..ac8caa70c0 100644
--- a/internal/http/services/owncloud/ocdav/copy.go
+++ b/internal/http/services/owncloud/ocdav/copy.go
@@ -237,7 +237,7 @@ func (s *svc) executePathCopy(ctx context.Context, client gateway.GatewayAPIClie
 
 		// 3. do download
 
-		httpDownloadReq, err := rhttp.NewRequest(ctx, "GET", downloadEP, nil)
+		httpDownloadReq, err := rhttp.NewRequest(ctx, http.MethodGet, downloadEP, nil)
 		if err != nil {
 			return err
 		}
@@ -254,7 +254,7 @@ func (s *svc) executePathCopy(ctx context.Context, client gateway.GatewayAPIClie
 
 		// 4. do upload
 
-		httpUploadReq, err := rhttp.NewRequest(ctx, "PUT", uploadEP, httpDownloadRes.Body)
+		httpUploadReq, err := rhttp.NewRequest(ctx, http.MethodPut, uploadEP, httpDownloadRes.Body)
 		if err != nil {
 			return err
 		}
diff --git a/internal/http/services/owncloud/ocdav/tpc.go b/internal/http/services/owncloud/ocdav/tpc.go
index c31b6066a4..aa324064bf 100644
--- a/internal/http/services/owncloud/ocdav/tpc.go
+++ b/internal/http/services/owncloud/ocdav/tpc.go
@@ -178,7 +178,7 @@ func (s *svc) performHTTPPull(ctx context.Context, client gateway.GatewayAPIClie
 	// get http client for remote
 	httpClient := &http.Client{}
 
-	req, err := http.NewRequest("GET", src, nil)
+	req, err := http.NewRequest(http.MethodGet, src, nil)
 	if err != nil {
 		w.WriteHeader(http.StatusInternalServerError)
 		return err
@@ -240,7 +240,7 @@ func (s *svc) performHTTPPull(ctx context.Context, client gateway.GatewayAPIClie
 	tempReader := io.TeeReader(httpDownloadRes.Body, &wc)
 
 	// do Upload
-	httpUploadReq, err := rhttp.NewRequest(ctx, "PUT", uploadEP, tempReader)
+	httpUploadReq, err := rhttp.NewRequest(ctx, http.MethodPut, uploadEP, tempReader)
 	if err != nil {
 		w.WriteHeader(http.StatusInternalServerError)
 		return err
@@ -362,7 +362,7 @@ func (s *svc) performHTTPPush(ctx context.Context, client gateway.GatewayAPIClie
 	}
 
 	// do download
-	httpDownloadReq, err := rhttp.NewRequest(ctx, "GET", downloadEP, nil)
+	httpDownloadReq, err := rhttp.NewRequest(ctx, http.MethodGet, downloadEP, nil)
 	if err != nil {
 		w.WriteHeader(http.StatusInternalServerError)
 		return err
@@ -387,7 +387,7 @@ func (s *svc) performHTTPPush(ctx context.Context, client gateway.GatewayAPIClie
 
 	// get http client for a remote call
 	httpClient := &http.Client{}
-	req, err := http.NewRequest("PUT", dst, tempReader)
+	req, err := http.NewRequest(http.MethodPut, dst, tempReader)
 	if err != nil {
 		w.WriteHeader(http.StatusInternalServerError)
 		return err
diff --git a/internal/http/services/owncloud/ocs/cache.go b/internal/http/services/owncloud/ocs/cache.go
index 8ab887d4c6..3b82135110 100644
--- a/internal/http/services/owncloud/ocs/cache.go
+++ b/internal/http/services/owncloud/ocs/cache.go
@@ -48,7 +48,7 @@ func (s *svc) cacheWarmup(w http.ResponseWriter, r *http.Request) {
 		ctx = ctxpkg.ContextSetToken(ctx, tkn)
 		ctx = metadata.AppendToOutgoingContext(ctx, ctxpkg.TokenHeader, tkn)
 
-		req, _ := http.NewRequest("GET", "", nil)
+		req, _ := http.NewRequest(http.MethodGet, "", nil)
 		req = req.WithContext(ctx)
 		req.URL = r.URL
 
diff --git a/pkg/app/provider/demo/demo.go b/pkg/app/provider/demo/demo.go
index 9376251aa1..ce36738188 100644
--- a/pkg/app/provider/demo/demo.go
+++ b/pkg/app/provider/demo/demo.go
@@ -21,6 +21,7 @@ package demo
 import (
 	"context"
 	"fmt"
+	"net/http"
 
 	appprovider "github.com/cs3org/go-cs3apis/cs3/app/provider/v1beta1"
 	appregistry "github.com/cs3org/go-cs3apis/cs3/app/registry/v1beta1"
@@ -42,7 +43,7 @@ func (p *demoProvider) GetAppURL(ctx context.Context, resource *provider.Resourc
 	url := fmt.Sprintf("<iframe src=%s/open/%s?view-mode=%s&access-token=%s&lang=%s />", p.iframeUIProvider, resource.Id.StorageId+":"+resource.Id.OpaqueId, viewMode.String(), token, language)
 	return &appprovider.OpenInAppURL{
 		AppUrl: url,
-		Method: "GET",
+		Method: http.MethodGet,
 	}, nil
 }
 
diff --git a/pkg/app/provider/wopi/wopi.go b/pkg/app/provider/wopi/wopi.go
index 1e172f16e1..7f0d096dc4 100644
--- a/pkg/app/provider/wopi/wopi.go
+++ b/pkg/app/provider/wopi/wopi.go
@@ -128,7 +128,7 @@ func (p *wopiProvider) GetAppURL(ctx context.Context, resource *provider.Resourc
 	}
 	wopiurl.Path = path.Join(wopiurl.Path, "/wopi/iop/openinapp")
 
-	httpReq, err := rhttp.NewRequest(ctx, "GET", wopiurl.String(), nil)
+	httpReq, err := rhttp.NewRequest(ctx, http.MethodGet, wopiurl.String(), nil)
 	if err != nil {
 		return nil, err
 	}
@@ -247,14 +247,14 @@ func (p *wopiProvider) GetAppURL(ctx context.Context, resource *provider.Resourc
 	// Depending on whether the WOPI server returned any form parameters or not,
 	// we decide whether the request method is POST or GET
 	var formParams map[string]string
-	method := "GET"
+	method := http.MethodGet
 	if form, ok := result["form-parameters"].(map[string]interface{}); ok {
 		if tkn, ok := form["access_token"].(string); ok {
 			formParams = map[string]string{
 				"access_token":     tkn,
 				"access_token_ttl": tokenTTL,
 			}
-			method = "POST"
+			method = http.MethodPost
 		}
 	}
 
@@ -302,7 +302,7 @@ func getAppURLs(c *config) (map[string]map[string]string, error) {
 	}
 	appurl.Path = path.Join(appurl.Path, "/hosting/discovery")
 
-	discReq, err := http.NewRequest("GET", appurl.String(), nil)
+	discReq, err := http.NewRequest(http.MethodGet, appurl.String(), nil)
 	if err != nil {
 		return nil, err
 	}
@@ -321,7 +321,7 @@ func getAppURLs(c *config) (map[string]map[string]string, error) {
 		}
 	} else if discRes.StatusCode == http.StatusNotFound {
 		// this may be a bridge-supported app
-		discReq, err = http.NewRequest("GET", c.AppIntURL, nil)
+		discReq, err = http.NewRequest(http.MethodGet, c.AppIntURL, nil)
 		if err != nil {
 			return nil, err
 		}
diff --git a/pkg/cbox/utils/tokenmanagement.go b/pkg/cbox/utils/tokenmanagement.go
index 273411e79e..f4bbe2241f 100644
--- a/pkg/cbox/utils/tokenmanagement.go
+++ b/pkg/cbox/utils/tokenmanagement.go
@@ -96,7 +96,7 @@ func (a *APITokenManager) getAPIToken(ctx context.Context) (string, time.Time, e
 		"audience":   {a.conf.TargetAPI},
 	}
 
-	httpReq, err := http.NewRequest("POST", a.conf.OIDCTokenEndpoint, strings.NewReader(params.Encode()))
+	httpReq, err := http.NewRequest(http.MethodPost, a.conf.OIDCTokenEndpoint, strings.NewReader(params.Encode()))
 	if err != nil {
 		return "", time.Time{}, err
 	}
@@ -135,7 +135,7 @@ func (a *APITokenManager) SendAPIGetRequest(ctx context.Context, url string, for
 		return nil, err
 	}
 
-	httpReq, err := http.NewRequest("GET", url, nil)
+	httpReq, err := http.NewRequest(http.MethodGet, url, nil)
 	if err != nil {
 		return nil, err
 	}
diff --git a/pkg/datatx/manager/rclone/rclone.go b/pkg/datatx/manager/rclone/rclone.go
index 41752ed6ed..6365ffa6a4 100644
--- a/pkg/datatx/manager/rclone/rclone.go
+++ b/pkg/datatx/manager/rclone/rclone.go
@@ -332,7 +332,7 @@ func (driver *rclone) startJob(ctx context.Context, transferID string, srcRemote
 	}
 	u.Path = path.Join(u.Path, transferFileMethod)
 	requestURL := u.String()
-	req, err := http.NewRequest("POST", requestURL, bytes.NewReader(data))
+	req, err := http.NewRequest(http.MethodPost, requestURL, bytes.NewReader(data))
 	if err != nil {
 		err = errors.Wrap(err, "rclone: error pulling transfer: error framing post request")
 		transfer.TransferStatus = datatx.Status_STATUS_TRANSFER_FAILED
@@ -474,7 +474,7 @@ func (driver *rclone) startJob(ctx context.Context, transferID string, srcRemote
 			u.Path = path.Join(u.Path, transferFileMethod)
 			requestURL := u.String()
 
-			req, err := http.NewRequest("POST", requestURL, bytes.NewReader(data))
+			req, err := http.NewRequest(http.MethodPost, requestURL, bytes.NewReader(data))
 			if err != nil {
 				logger.Error().Err(err).Msgf("rclone driver: error framing post request: %v", err)
 				transfer.TransferStatus = datatx.Status_STATUS_INVALID
@@ -653,7 +653,7 @@ func (driver *rclone) CancelTransfer(ctx context.Context, transferID string) (*d
 	u.Path = path.Join(u.Path, transferFileMethod)
 	requestURL := u.String()
 
-	req, err := http.NewRequest("POST", requestURL, bytes.NewReader(data))
+	req, err := http.NewRequest(http.MethodPost, requestURL, bytes.NewReader(data))
 	if err != nil {
 		err = errors.Wrap(err, "rclone driver: error framing post request")
 		return &datatx.TxInfo{
@@ -781,7 +781,7 @@ func (driver *rclone) remotePathIsFolder(remote string, remotePath string, remot
 	u.Path = path.Join(u.Path, listMethod)
 	requestURL := u.String()
 
-	req, err := http.NewRequest("POST", requestURL, bytes.NewReader(data))
+	req, err := http.NewRequest(http.MethodPost, requestURL, bytes.NewReader(data))
 	if err != nil {
 		return false, errors.Wrap(err, "rclone driver: error framing post request")
 	}
diff --git a/pkg/eosclient/eosgrpc/eoshttp.go b/pkg/eosclient/eosgrpc/eoshttp.go
index e7631b8951..1ec18a804e 100644
--- a/pkg/eosclient/eosgrpc/eoshttp.go
+++ b/pkg/eosclient/eosgrpc/eoshttp.go
@@ -208,16 +208,15 @@ func (c *EOSHTTPClient) getRespError(rsp *http.Response, err error) error {
 	}
 
 	switch rsp.StatusCode {
-	case 0, 200, 201:
+	case 0, http.StatusOK, http.StatusCreated:
 		return nil
-	case 403:
+	case http.StatusForbidden:
 		return errtypes.PermissionDenied(rspdesc(rsp))
-	case 404:
+	case http.StatusNotFound:
 		return errtypes.NotFound(rspdesc(rsp))
 	}
 
-	err2 := errtypes.InternalError("Err from EOS: " + rspdesc(rsp))
-	return err2
+	return errtypes.InternalError("Err from EOS: " + rspdesc(rsp))
 }
 
 // From the basepath and the file path... build an url
@@ -262,7 +261,7 @@ func (c *EOSHTTPClient) GETFile(ctx context.Context, remoteuser string, auth eos
 		log.Error().Str("func", "GETFile").Str("url", finalurl).Str("err", err.Error()).Msg("can't create request")
 		return nil, err
 	}
-	req, err := http.NewRequestWithContext(ctx, "GET", finalurl, nil)
+	req, err := http.NewRequestWithContext(ctx, http.MethodGet, finalurl, nil)
 	if err != nil {
 		log.Error().Str("func", "GETFile").Str("url", finalurl).Str("err", err.Error()).Msg("can't create request")
 		return nil, err
@@ -298,7 +297,7 @@ func (c *EOSHTTPClient) GETFile(ctx context.Context, remoteuser string, auth eos
 				return nil, err
 			}
 
-			req, err = http.NewRequestWithContext(ctx, "GET", loc.String(), nil)
+			req, err = http.NewRequestWithContext(ctx, http.MethodGet, loc.String(), nil)
 			if err != nil {
 				log.Error().Str("func", "GETFile").Str("url", loc.String()).Str("err", err.Error()).Msg("can't create redirected request")
 				return nil, err
@@ -354,7 +353,7 @@ func (c *EOSHTTPClient) PUTFile(ctx context.Context, remoteuser string, auth eos
 		log.Error().Str("func", "PUTFile").Str("url", finalurl).Str("err", err.Error()).Msg("can't create request")
 		return err
 	}
-	req, err := http.NewRequestWithContext(ctx, "PUT", finalurl, nil)
+	req, err := http.NewRequestWithContext(ctx, http.MethodPut, finalurl, nil)
 	if err != nil {
 		log.Error().Str("func", "PUTFile").Str("url", finalurl).Str("err", err.Error()).Msg("can't create request")
 		return err
@@ -392,7 +391,7 @@ func (c *EOSHTTPClient) PUTFile(ctx context.Context, remoteuser string, auth eos
 				return err
 			}
 
-			req, err = http.NewRequestWithContext(ctx, "PUT", loc.String(), stream)
+			req, err = http.NewRequestWithContext(ctx, http.MethodPut, loc.String(), stream)
 			if err != nil {
 				log.Error().Str("func", "PUTFile").Str("url", loc.String()).Str("err", err.Error()).Msg("can't create redirected request")
 				return err
@@ -454,7 +453,7 @@ func (c *EOSHTTPClient) Head(ctx context.Context, remoteuser string, auth eoscli
 		return err
 	}
 
-	req, err := http.NewRequestWithContext(ctx, "HEAD", finalurl, nil)
+	req, err := http.NewRequestWithContext(ctx, http.MethodHead, finalurl, nil)
 	if err != nil {
 		log.Error().Str("func", "Head").Str("remoteuser", remoteuser).Str("uid,gid", auth.Role.UID+","+auth.Role.GID).Str("url", finalurl).Str("err", err.Error()).Msg("can't create request")
 		return err
diff --git a/pkg/metrics/driver/xcloud/xcloud.go b/pkg/metrics/driver/xcloud/xcloud.go
index d469551650..553c68b146 100644
--- a/pkg/metrics/driver/xcloud/xcloud.go
+++ b/pkg/metrics/driver/xcloud/xcloud.go
@@ -62,7 +62,7 @@ func (d *CloudDriver) refresh() error {
 	// endpoint example: https://mybox.com or https://mybox.com/owncloud
 	endpoint := fmt.Sprintf("%s/index.php/apps/sciencemesh/internal_metrics", d.instance)
 
-	req, err := http.NewRequest("GET", endpoint, nil)
+	req, err := http.NewRequest(http.MethodGet, endpoint, nil)
 	if err != nil {
 		log.Err(err).Msgf("xcloud: error creating request to %s", d.instance)
 		return err
diff --git a/pkg/ocm/invite/manager/json/json.go b/pkg/ocm/invite/manager/json/json.go
index 9849e245d9..d82c0ad6d9 100644
--- a/pkg/ocm/invite/manager/json/json.go
+++ b/pkg/ocm/invite/manager/json/json.go
@@ -221,7 +221,7 @@ func (m *manager) ForwardInvite(ctx context.Context, invite *invitepb.InviteToke
 	u.Path = path.Join(u.Path, acceptInviteEndpoint)
 	recipientURL := u.String()
 
-	req, err := http.NewRequest("POST", recipientURL, strings.NewReader(requestBody.Encode()))
+	req, err := http.NewRequest(http.MethodPost, recipientURL, strings.NewReader(requestBody.Encode()))
 	if err != nil {
 		return errors.Wrap(err, "json: error framing post request")
 	}
diff --git a/pkg/ocm/invite/manager/memory/memory.go b/pkg/ocm/invite/manager/memory/memory.go
index bf7f8b22fa..f5a6ef2cb4 100644
--- a/pkg/ocm/invite/manager/memory/memory.go
+++ b/pkg/ocm/invite/manager/memory/memory.go
@@ -119,7 +119,7 @@ func (m *manager) ForwardInvite(ctx context.Context, invite *invitepb.InviteToke
 	u.Path = path.Join(u.Path, acceptInviteEndpoint)
 	recipientURL := u.String()
 
-	req, err := http.NewRequest("POST", recipientURL, strings.NewReader(requestBody.Encode()))
+	req, err := http.NewRequest(http.MethodPost, recipientURL, strings.NewReader(requestBody.Encode()))
 	if err != nil {
 		return errors.Wrap(err, "json: error framing post request")
 	}
diff --git a/pkg/ocm/provider/authorizer/mentix/mentix.go b/pkg/ocm/provider/authorizer/mentix/mentix.go
index 30ab53c0ef..3ac345ee77 100644
--- a/pkg/ocm/provider/authorizer/mentix/mentix.go
+++ b/pkg/ocm/provider/authorizer/mentix/mentix.go
@@ -101,7 +101,7 @@ func (a *authorizer) fetchProviders() ([]*ocmprovider.ProviderInfo, error) {
 		return a.providers, nil
 	}
 
-	req, err := http.NewRequest("GET", a.client.BaseURL, nil)
+	req, err := http.NewRequest(http.MethodGet, a.client.BaseURL, nil)
 	if err != nil {
 		return nil, err
 	}
diff --git a/pkg/ocm/share/sender/sender.go b/pkg/ocm/share/sender/sender.go
index 32e414e93d..8f53f241a4 100644
--- a/pkg/ocm/share/sender/sender.go
+++ b/pkg/ocm/share/sender/sender.go
@@ -64,7 +64,7 @@ func Send(requestBodyMap map[string]interface{}, pi *ocmprovider.ProviderInfo) e
 	u.Path = path.Join(u.Path, createOCMCoreShareEndpoint)
 	recipientURL := u.String()
 
-	req, err := http.NewRequest("POST", recipientURL, strings.NewReader(string(requestBody)))
+	req, err := http.NewRequest(http.MethodPost, recipientURL, strings.NewReader(string(requestBody)))
 	if err != nil {
 		return errors.Wrap(err, "sender: error framing post request")
 	}
diff --git a/pkg/rhttp/datatx/manager/simple/simple.go b/pkg/rhttp/datatx/manager/simple/simple.go
index 21bfde8244..b1e49a0bad 100644
--- a/pkg/rhttp/datatx/manager/simple/simple.go
+++ b/pkg/rhttp/datatx/manager/simple/simple.go
@@ -67,9 +67,9 @@ func (m *manager) Handler(fs storage.FS) (http.Handler, error) {
 		sublog := appctx.GetLogger(ctx).With().Str("datatx", "simple").Logger()
 
 		switch r.Method {
-		case "GET", "HEAD":
+		case http.MethodGet, http.MethodHead:
 			download.GetOrHeadFile(w, r, fs, "")
-		case "PUT":
+		case http.MethodPut:
 			fn := r.URL.Path
 			defer r.Body.Close()
 
diff --git a/pkg/rhttp/datatx/manager/spaces/spaces.go b/pkg/rhttp/datatx/manager/spaces/spaces.go
index bb982c7a85..0e9057659d 100644
--- a/pkg/rhttp/datatx/manager/spaces/spaces.go
+++ b/pkg/rhttp/datatx/manager/spaces/spaces.go
@@ -75,9 +75,9 @@ func (m *manager) Handler(fs storage.FS) (http.Handler, error) {
 		sublog := appctx.GetLogger(ctx).With().Str("datatx", "spaces").Str("space", spaceID).Logger()
 
 		switch r.Method {
-		case "GET", "HEAD":
+		case http.MethodGet, http.MethodHead:
 			download.GetOrHeadFile(w, r, fs, spaceID)
-		case "PUT":
+		case http.MethodPut:
 			// make a clean relative path
 			fn := path.Clean(strings.TrimLeft(r.URL.Path, "/"))
 			defer r.Body.Close()
diff --git a/pkg/rhttp/datatx/manager/tus/tus.go b/pkg/rhttp/datatx/manager/tus/tus.go
index 7ece3466fb..3e442417ab 100644
--- a/pkg/rhttp/datatx/manager/tus/tus.go
+++ b/pkg/rhttp/datatx/manager/tus/tus.go
@@ -94,15 +94,15 @@ func (m *manager) Handler(fs storage.FS) (http.Handler, error) {
 		}
 
 		switch method {
-		case "POST":
+		case http.MethodPost:
 			handler.PostFile(w, r)
-		case "HEAD":
+		case http.MethodHead:
 			handler.HeadFile(w, r)
-		case "PATCH":
+		case http.MethodPatch:
 			handler.PatchFile(w, r)
 		case "DELETE":
 			handler.DelFile(w, r)
-		case "GET":
+		case http.MethodGet:
 			download.GetOrHeadFile(w, r, fs, "")
 		default:
 			w.WriteHeader(http.StatusNotImplemented)
diff --git a/pkg/rhttp/datatx/utils/download/download.go b/pkg/rhttp/datatx/utils/download/download.go
index f96e9d0328..9d62531ca7 100644
--- a/pkg/rhttp/datatx/utils/download/download.go
+++ b/pkg/rhttp/datatx/utils/download/download.go
@@ -184,7 +184,7 @@ func GetOrHeadFile(w http.ResponseWriter, r *http.Request, fs storage.FS, spaceI
 
 	w.WriteHeader(code)
 
-	if r.Method != "HEAD" {
+	if r.Method != http.MethodHead {
 		var c int64
 		c, err = io.CopyN(w, sendContent, sendSize)
 		if err != nil {
diff --git a/pkg/sdk/action/download.go b/pkg/sdk/action/download.go
index b80668cbb4..4cabeb5158 100644
--- a/pkg/sdk/action/download.go
+++ b/pkg/sdk/action/download.go
@@ -20,6 +20,7 @@ package action
 
 import (
 	"fmt"
+	"net/http"
 
 	gateway "github.com/cs3org/go-cs3apis/cs3/gateway/v1beta1"
 	provider "github.com/cs3org/go-cs3apis/cs3/storage/provider/v1beta1"
@@ -75,7 +76,7 @@ func (action *DownloadAction) Download(fileInfo *storage.ResourceInfo) ([]byte,
 	}
 
 	// WebDAV is not supported, so directly read the HTTP endpoint
-	request, err := action.session.NewHTTPRequest(p.DownloadEndpoint, "GET", p.Token, nil)
+	request, err := action.session.NewHTTPRequest(p.DownloadEndpoint, http.MethodGet, p.Token, nil)
 	if err != nil {
 		return nil, fmt.Errorf("unable to create an HTTP request for '%v': %v", p.DownloadEndpoint, err)
 	}
diff --git a/pkg/sdk/action/upload.go b/pkg/sdk/action/upload.go
index 263722d670..97c3363305 100644
--- a/pkg/sdk/action/upload.go
+++ b/pkg/sdk/action/upload.go
@@ -23,6 +23,7 @@ import (
 	"fmt"
 	"io"
 	"math"
+	"net/http"
 	"os"
 	p "path"
 	"strconv"
@@ -171,7 +172,7 @@ func (action *UploadAction) selectChecksumType(checksumTypes []*provider.Resourc
 }
 
 func (action *UploadAction) uploadFilePUT(upload *gateway.FileUploadProtocol, data io.Reader, checksum string, checksumType string) error {
-	request, err := action.session.NewHTTPRequest(upload.UploadEndpoint, "PUT", upload.Token, data)
+	request, err := action.session.NewHTTPRequest(upload.UploadEndpoint, http.MethodPut, upload.Token, data)
 	if err != nil {
 		return fmt.Errorf("unable to create HTTP request for '%v': %v", upload.UploadEndpoint, err)
 	}
diff --git a/pkg/sdk/common/net/tus.go b/pkg/sdk/common/net/tus.go
index bf4a73df3c..0f9db857d4 100644
--- a/pkg/sdk/common/net/tus.go
+++ b/pkg/sdk/common/net/tus.go
@@ -74,7 +74,7 @@ func (client *TUSClient) checkEndpointCreationOption(endpoint string) bool {
 		Timeout: time.Duration(1.5 * float64(time.Second)),
 	}
 
-	if httpReq, err := http.NewRequest("OPTIONS", endpoint, nil); err == nil {
+	if httpReq, err := http.NewRequest(http.MethodOptions, endpoint, nil); err == nil {
 		if res, err := httpClient.Do(httpReq); err == nil {
 			defer res.Body.Close()
 
diff --git a/pkg/siteacc/manager/gocdb/account.go b/pkg/siteacc/manager/gocdb/account.go
index a90d606a76..93780efe6c 100644
--- a/pkg/siteacc/manager/gocdb/account.go
+++ b/pkg/siteacc/manager/gocdb/account.go
@@ -77,7 +77,7 @@ func writeAccount(account *data.Account, operation string, address string, apiKe
 	}
 	defer resp.Body.Close()
 
-	if resp.StatusCode >= 400 {
+	if resp.StatusCode >= http.StatusBadRequest {
 		msg, _ := io.ReadAll(resp.Body)
 		return errors.Errorf("unable to perform request: %v", string(msg))
 	}
diff --git a/pkg/storage/fs/cback/cback.go b/pkg/storage/fs/cback/cback.go
index 63d2c162bb..d1c6128fda 100644
--- a/pkg/storage/fs/cback/cback.go
+++ b/pkg/storage/fs/cback/cback.go
@@ -337,7 +337,7 @@ func (fs *cback) Download(ctx context.Context, ref *provider.Reference) (io.Read
 		ssID, searchPath = fs.pathTrimmer(snapshotList, resp)
 
 		url := fs.conf.APIURL + "/backups/" + strconv.Itoa(resp.ID) + "/snapshots/" + ssID + "/" + searchPath
-		requestType := "GET"
+		requestType := http.MethodGet
 		md, err := fs.GetMD(ctx, ref, nil)
 
 		if err != nil {
diff --git a/pkg/storage/fs/nextcloud/nextcloud.go b/pkg/storage/fs/nextcloud/nextcloud.go
index 71c3617373..3621b468e0 100644
--- a/pkg/storage/fs/nextcloud/nextcloud.go
+++ b/pkg/storage/fs/nextcloud/nextcloud.go
@@ -169,7 +169,7 @@ func (nc *StorageDriver) doDownload(ctx context.Context, filePath string) (io.Re
 	if err != nil {
 		panic(err)
 	}
-	if resp.StatusCode != 200 {
+	if resp.StatusCode != http.StatusOK {
 		panic("No 200 response code in download request")
 	}
 
@@ -193,7 +193,7 @@ func (nc *StorageDriver) doDownloadRevision(ctx context.Context, filePath string
 	if err != nil {
 		panic(err)
 	}
-	if resp.StatusCode != 200 {
+	if resp.StatusCode != http.StatusOK {
 		panic("No 200 response code in download request")
 	}