fix gosec issues

dell · May 10, 2024 · 8dfec37 · 8dfec37
1 parent a16c042
commit 8dfec37
Show file tree

Hide file tree

Showing 6 changed files with 86 additions and 86 deletions.
diff --git a/.github/workflows/actions.yml b/.github/workflows/actions.yml
@@ -15,7 +15,7 @@ jobs:
         uses: securego/gosec@master
         with:
           # added additional exclude arguments after gosec v2.9.4 came out
-          args: -exclude=G402,G101 ./...
+          args: -exclude=G101,G402 ./...
   malware_security_scan:
     name: Malware Scanner
     runs-on: ubuntu-latest

diff --git a/api/api.go b/api/api.go
@@ -284,7 +284,7 @@ func New(
 	}
 
 	if c.authType == authTypeSessionBased {
-		c.authenticate(ctx, username, password, hostname)
+		_ = c.authenticate(ctx, username, password, hostname)
 	}
 	resp := &apiVerResponse{}
 	if err := c.Get(ctx, "/platform/latest", "", nil, nil, resp); err != nil &&

diff --git a/api/api_logging.go b/api/api_logging.go
@@ -47,7 +47,7 @@ func logRequest(_ context.Context, w io.Writer, req *http.Request, verbose Verbo
 		// full logging, i.e. print full request message content
 		buf, _ := httputil.DumpRequest(req, !isBinOctetBody(req.Header))
 		decodedBuf := encryptPassword(buf)
-		WriteIndented(w, decodedBuf)
+		_ = WriteIndented(w, decodedBuf)
 		fmt.Fprintln(w)
 	}
 }
@@ -75,7 +75,7 @@ func logResponse(ctx context.Context, res *http.Response, verbose VerboseType) {
 	}
 
 	// when DumpResponse gets err, buf will be nil. No message content will be printed
-	WriteIndented(w, buf)
+	_ = WriteIndented(w, buf)
 
 	log.Debug(ctx, w.String())
 }

diff --git a/api/api_ordered_values.go b/api/api_ordered_values.go
@@ -201,7 +201,7 @@ func (v *OrderedValues) Del(key []byte) {
 // using insertion order.
 func (v *OrderedValues) Encode() string {
 	buf := &bytes.Buffer{}
-	v.EncodeTo(buf)
+	_ = v.EncodeTo(buf)
 	return buf.String()
 }