fix linter issues

Author: cx-leonardo-fontes

engine/engine.go

@@ -290,10 +290,10 @@ func GetRulesCommand(engineConfig *EngineConfig) *cobra.Command {
 
 			tab := tabwriter.NewWriter(os.Stdout, 1, 2, 2, ' ', 0)
 
-			fmt.Fprintln(tab, "Name\tDescription\tTags\tValidity Check")
-			fmt.Fprintln(tab, "----\t----\t----\t----")
+			fmt.Fprintln(tab, "Name\tDescription\tTags\tValidity Check") //nolint:errcheck
+			fmt.Fprintln(tab, "----\t----\t----\t----")                  //nolint:errcheck
 			for _, rule := range *rules {
-				fmt.Fprintf(
+				fmt.Fprintf( //nolint:errcheck
 					tab,
 					"%s\t%s\t%s\t%s\n",
 					rule.Rule.RuleID,
@@ -352,7 +352,8 @@ func getStartAndEndLines(ctx context.Context, pluginName string, gitInfo *plugin
 	var startLine, endLine int
 	var err error
 
-	if pluginName == "filesystem" {
+	switch pluginName {
+	case "filesystem":
 		totalLines, totalOK := ctx.Value(totalLinesKey).(int)
 		chunkLines, chunkOK := ctx.Value(linesInChunkKey).(int)
 
@@ -363,12 +364,12 @@ func getStartAndEndLines(ctx context.Context, pluginName string, gitInfo *plugin
 
 		startLine = value.StartLine + offset
 		endLine = value.EndLine + offset
-	} else if pluginName == "git" {
+	case "git":
 		startLine, endLine, err = plugins.GetGitStartAndEndLine(gitInfo, value.StartLine, value.EndLine)
 		if err != nil {
 			return 0, 0, err
 		}
-	} else {
+	default:
 		startLine = value.StartLine
 		endLine = value.EndLine
 	}

engine/engine_test.go

@@ -470,7 +470,7 @@ func writeTempFile(t *testing.T, dir string, size int, content []byte) string {
 
 	f, err := os.CreateTemp(dir, "testfile-*.tmp")
 	require.NoError(t, err, "create temp file")
-	defer f.Close()
+	defer f.Close() //nolint:errcheck
 
 	var data []byte
 	if content != nil {

lib/reporting/report_test.go

@@ -221,8 +221,6 @@ func TestWriteReportInNonExistingDir(t *testing.T) {
 	if err != nil {
 		t.Error(err)
 	}
-
-	os.RemoveAll(filepath.Join(tempDir, "test_temp_dir"))
 }
 
 func TestGetOutputSarif(t *testing.T) {

lib/utils/flags_test.go

@@ -685,7 +685,7 @@ subcommand:
 			var configFileName string
 			if tc.config != nil {
 				configFileName = writeTempFile(t, tc.config, tc.configFormat)
-				defer os.Remove(configFileName)
+				defer os.Remove(configFileName) //nolint:errcheck
 
 				tc.args = append(tc.args, "--"+configFlagName, configFileName)
 			}

lib/utils/http.go

@@ -43,7 +43,7 @@ func HttpRequest(method string, url string, authorization IAuthorizationHeader,
 		return nil, response, fmt.Errorf("unable to send http request %w", err)
 	}
 
-	defer response.Body.Close()
+	defer response.Body.Close() //nolint:errcheck
 
 	if response.StatusCode < 200 || response.StatusCode >= 300 {
 		if retry.MaxRetries > 0 {

plugins/git.go

@@ -58,7 +58,7 @@ func (p *GitPlugin) DefineCommand(items chan ISourceItem, errors chan error) (*c
 		Args:  cobra.MatchAll(cobra.ExactArgs(1), validGitRepoArgs),
 		Run: func(cmd *cobra.Command, args []string) {
 			log.Info().Msg("Git plugin started")
-			p.scanGit(args[0], p.buildScanOptions(), p.Channels.Items, p.Channels.Errors)
+			p.scanGit(args[0], p.buildScanOptions(), p.Items, p.Errors)
 			p.WaitGroup.Wait()
 			close(items)
 		},

plugins/paligo.go

@@ -73,7 +73,7 @@ func (p *PaligoPlugin) DefineCommand(items chan ISourceItem, errors chan error)
 		Run: func(cmd *cobra.Command, args []string) {
 			// Waits for MarkFlagsOneRequired https://github.com/spf13/cobra/pull/1952
 			if p.auth == "" && (p.username == "" || p.token == "") {
-				p.Channels.Errors <- fmt.Errorf("exactly one of the flags in the group %v must be set; none were set", []string{paligoAuthFlag, paligoUsernameFlag, paligoTokenFlag})
+				p.Errors <- fmt.Errorf("exactly one of the flags in the group %v must be set; none were set", []string{paligoAuthFlag, paligoUsernameFlag, paligoTokenFlag})
 				return
 			}
 			log.Info().Msg("Paligo plugin started")
@@ -107,7 +107,7 @@ func (p *PaligoPlugin) getItems() {
 
 	foldersToProcess, err := p.getFirstProcessingFolders()
 	if err != nil {
-		p.Channels.Errors <- err
+		p.Errors <- err
 		return
 	}
 
@@ -156,14 +156,15 @@ func (p *PaligoPlugin) processFolders(foldersToProcess []PaligoItem) chan Paligo
 			folderInfo, err := p.paligoApi.showFolder(folder.ID)
 			if err != nil {
 				log.Error().Err(err).Msgf("error while getting %s '%s'", folder.Type, folder.Name)
-				p.Channels.Errors <- err
+				p.Errors <- err
 				continue
 			}
 
 			for _, child := range folderInfo.Children {
-				if child.Type == "component" {
+				switch child.Type {
+				case "component":
 					itemsChan <- child
-				} else if child.Type == "folder" {
+				case "folder":
 					foldersToProcess = append(foldersToProcess, child)
 				}
 			}
@@ -180,7 +181,7 @@ func (p *PaligoPlugin) handleComponent(paligoItem PaligoItem) {
 	document, err := p.paligoApi.showDocument(paligoItem.ID)
 	if err != nil {
 		log.Error().Err(err).Msgf("error while getting document '%s'", paligoItem.Name)
-		p.Channels.Errors <- fmt.Errorf("error while getting document '%s': %w", paligoItem.Name, err)
+		p.Errors <- fmt.Errorf("error while getting document '%s': %w", paligoItem.Name, err)
 		return
 	}
 

plugins/slack.go

@@ -52,7 +52,7 @@ func (p *SlackPlugin) DefineCommand(items chan ISourceItem, errors chan error) (
 		Long:  "Scan Slack team for sensitive information.",
 		Run: func(cmd *cobra.Command, args []string) {
 			p.getItems()
-			p.Channels.WaitGroup.Wait()
+			p.WaitGroup.Wait()
 			close(items)
 		},
 	}

tests/lint.go

@@ -66,7 +66,7 @@ func lintFile(path string) error {
 	if err != nil {
 		return err
 	}
-	defer file.Close()
+	defer file.Close() //nolint:errcheck
 
 	scanner := bufio.NewScanner(file)
 	line := 1