chore(lint): update golangci lint version #7445

cx-artur-ribeiro · 2025-04-21T11:51:11Z

Reason for Proposed Changes

Golangci lint version is outdated and may cause inconsistent results;

Proposed Changes

Update golangci lint version to 2.0.2;

I submit this contribution under the Apache-2.0 license.

github-actions · 2025-04-21T11:51:42Z

KICS version: v2.1.7

	Category	Results
	CRITICAL	0
	HIGH	0
	MEDIUM	0
	LOW	0
	INFO	0
	TRACE	0
	TOTAL	0

Metric	Values
Files scanned	1
Files parsed	1
Files failed to scan	0
Total executed queries	47
Queries failed to execute	0
Execution time	0

.github/scripts/report/e2e-html.go

@@ -78,7 +78,7 @@
 	fullPath := filepath.Join(path, filename)
 	t := template.Must(template.New("report.tmpl").Funcs(templateFuncs).Parse(htmlTemplate))

-	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


cmd/builder/main.go

@@ -67,7 +67,7 @@
 }

 func saveFile(filePath string, content []byte) error {
-	f, err := os.OpenFile(filepath.Clean(filePath), os.O_CREATE|os.O_WRONLY|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(filePath), os.O_CREATE|os.O_WRONLY|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/kuberneter/utils.go

@@ -40,7 +40,7 @@
 func (info *k8sAPICall) saveK8sResources(kind, k8sResourcesContent, apiVersionFolder string) {
 	file := filepath.Join(apiVersionFolder, kind+"s"+".yaml")

-	f, err := os.OpenFile(filepath.Clean(file), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(file), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/printer/options.go

@@ -90,12 +97,12 @@
 			return err
 		}
 	} else if filepath.Dir(logPath) != "." {
-		if createErr := os.MkdirAll(filepath.Dir(logPath), os.ModePerm); createErr != nil {
+		if createErr := os.MkdirAll(filepath.Dir(logPath), os.ModePerm); createErr != nil { //nolint:gosec


pkg/printer/options.go

 			return createErr
 		}
 	}

-	loggerFile, err = os.OpenFile(logPath, os.O_CREATE|os.O_WRONLY, os.ModePerm)
+	loggerFile, err = os.OpenFile(logPath, os.O_CREATE|os.O_WRONLY, os.ModePerm) //nolint:gosec


pkg/remediation/utils.go

@@ -58,7 +58,7 @@
 	filepath.Clean(originalFileName)
 	// create temporary file
 	tmpFile := filepath.Join(os.TempDir(), "temporary-remediation-"+utils.NextRandom()+"-"+filepath.Base(originalFileName))
-	f, err := os.OpenFile(tmpFile, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(tmpFile, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/remediation/utils.go

@@ -114,7 +114,7 @@
 func CreateTempFile(filePathCopyFrom, tmpFilePath string) string {
 	filepath.Clean(filePathCopyFrom)
 	filepath.Clean(tmpFilePath)
-	f, err := os.OpenFile(tmpFilePath, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(tmpFilePath, os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/report/commons.go

@@ -92,7 +92,7 @@
 	}
 	fullPath := filepath.Join(path, filename)

-	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/report/commons.go

@@ -124,7 +124,7 @@
 	}

 	fullPath := filepath.Join(path, filename)
-	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/report/html.go

@@ -92,7 +92,7 @@
 	fullPath := filepath.Join(path, filename)
 	t := template.Must(template.New("report.tmpl").Funcs(templateFuncs).Parse(htmlTemplate))

-	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


e2e/utils/csv.go

@@ -17,7 +17,7 @@
 	filePath := filepath.Join("output", filename)
 	fullPath := filepath.Join(cwd, filePath)

-	csvFile, err := os.Open(fullPath)
+	csvFile, err := os.Open(fullPath) //nolint:gosec


e2e/utils/html.go

@@ -30,7 +30,7 @@
 // HTMLValidation executes many asserts to validate the HTML Report
 func HTMLValidation(t *testing.T, file string) {
 	// Read & Parse Expected HTML Report
-	expectHTMLString, errExpStr := os.ReadFile(filepath.Join("fixtures", file))
+	expectHTMLString, errExpStr := os.ReadFile(filepath.Join("fixtures", file)) //nolint:gosec


internal/console/scan.go

@@ -81,7 +81,7 @@
 		if filepath.Ext(flags.GetStrFlag(flags.OutputPathFlag)) != "" {
 			flags.SetStrFlag(flags.OutputPathFlag, filepath.Join(flags.GetStrFlag(flags.OutputPathFlag), string(os.PathSeparator)))
 		}
-		if err := os.MkdirAll(flags.GetStrFlag(flags.OutputPathFlag), os.ModePerm); err != nil {
+		if err := os.MkdirAll(flags.GetStrFlag(flags.OutputPathFlag), os.ModePerm); err != nil { //nolint:gosec


e2e/utils/html.go

 	require.NoError(t, errExpStr, "Opening Expected HTML File should not yield an error")
 	expectedHTML, errExp := html.Parse(strings.NewReader(string(expectHTMLString)))
 	require.NoError(t, errExp, "Opening Expected HTML File should not yield an error")

 	// Read & Parse Output HTML Report
-	actualHTMLString, errActStr := os.ReadFile(filepath.Join("output", file))
+	actualHTMLString, errActStr := os.ReadFile(filepath.Join("output", file)) //nolint:gosec


internal/console/scan.go

 			return err
 		}
 	}
 	if flags.GetStrFlag(flags.PayloadPathFlag) != "" && filepath.Dir(flags.GetStrFlag(flags.PayloadPathFlag)) != "." {
-		if err := os.MkdirAll(filepath.Dir(flags.GetStrFlag(flags.PayloadPathFlag)), os.ModePerm); err != nil {
+		if err := os.MkdirAll(filepath.Dir(flags.GetStrFlag(flags.PayloadPathFlag)), os.ModePerm); err != nil { //nolint:gosec


pkg/engine/source/filesystem.go

@@ -134,7 +134,7 @@
 	}

 	if library != kicsDefault {
-		byteContent, err := os.ReadFile(library)
+		byteContent, err := os.ReadFile(library) //nolint:gosec


pkg/kuberneter/kuberneter.go

@@ -90,7 +90,7 @@

 	apiVersionFolder := filepath.Join(info.destinationPath, apiVersion)

-	if err := os.MkdirAll(apiVersionFolder, os.ModePerm); err != nil {
+	if err := os.MkdirAll(apiVersionFolder, os.ModePerm); err != nil { //nolint:gosec


pkg/kuberneter/utils.go

@@ -205,7 +205,7 @@
 	destFolderName := fmt.Sprintf("kics-extract-kuberneter-%s", time.Now().Format("01-02-2006"))
 	destination := filepath.Join(destinationPath, destFolderName)

-	if err := os.MkdirAll(destination, os.ModePerm); err != nil {
+	if err := os.MkdirAll(destination, os.ModePerm); err != nil { //nolint:gosec


pkg/parser/utils/certificate_utils.go

@@ -27,7 +27,7 @@
 }

 func getCertificateInfo(filePath string) (certInfo, error) {
-	certPEM, err := os.ReadFile(filePath)
+	certPEM, err := os.ReadFile(filePath) //nolint:gosec


pkg/report/commons.go

@@ -140,7 +140,7 @@

 func exportCSVReport(path, filename string, body []reportModel.CSVReport) error {
 	fullPath := filepath.Join(path, filename)
-	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm)
+	f, err := os.OpenFile(filepath.Clean(fullPath), os.O_WRONLY|os.O_CREATE|os.O_TRUNC, os.ModePerm) //nolint:gosec


pkg/engine/provider/filesystem.go

@@ -212,7 +212,7 @@
 		return nil, ErrNotSupportedFile
 	}

-	c, errOpenFile := os.Open(scanPath)
+	c, errOpenFile := os.Open(scanPath) //nolint:gosec


pkg/parser/terraform/terraform.go

@@ -152,7 +152,7 @@
 }

 func parseFile(filename string, shouldReplaceDataSource bool) (*hcl.File, error) {
-	file, err := os.ReadFile(filename)
+	file, err := os.ReadFile(filename) //nolint:gosec


pkg/scan/scan.go

@@ -202,7 +202,7 @@

 func getSecretsRegexRules(regexRulesPath string) (regexRulesContent string, err error) {
 	if regexRulesPath != "" {
-		b, err := os.ReadFile(regexRulesPath)
+		b, err := os.ReadFile(regexRulesPath) //nolint:gosec


internal/console/analyze.go

@@ -117,12 +117,16 @@
 		return err
 	}

-	f, err := os.Create(resultsPath)
+	f, err := os.Create(resultsPath) //nolint:gosec


…000-update-lint-version

update lint version to version 2.0.2

6528592

cx-artur-ribeiro added the CI PR related with GitHub Actions CI process label Apr 21, 2025

cx-artur-ribeiro self-assigned this Apr 21, 2025

update deprecated ubuntu version from 20.04 to latest

067eb07

cx-artur-ribeiro changed the title ~~build(lint): update lint version to version 2.0.2~~ ci(lint): update golanci lint version Apr 21, 2025

cx-artur-ribeiro added 3 commits April 21, 2025 14:04

fix lint issues

dd0f864

Merge branch 'master' into AST-0000-update-lint-version

b5ebe52

fix lint issues

6d6afbf