fix: modify code to be compliant to lint pipeline

alegrey91 · Jan 21, 2023 · 2ad08e4 · 2ad08e4
1 parent 2619487
commit 2ad08e4
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/cmd/daemon.go b/cmd/daemon.go
@@ -16,6 +16,7 @@ limitations under the License.
 package cmd
 
 import (
+	"fmt"
 
 	"github.com/spf13/cobra"
 )
@@ -25,7 +26,10 @@ var daemonCmd = &cobra.Command{
 	Use:   "daemon",
 	Short: "Run fwdctl as deamon",
 	Run: func(cmd *cobra.Command, args []string) {
-		cmd.Help()
+		err := cmd.Help()
+		if err != nil {
+			fmt.Println(err)
+		}
 	},
 }
 

diff --git a/internal/daemon/management.go b/internal/daemon/management.go
@@ -52,7 +52,9 @@ func Start(rulesFile string) {
 		errorLogger.Println(err)
 		os.Exit(1)
 	}
-	defer removePidFile()
+	defer func() {
+		err = removePidFile()
+	}()
 	infoLogger.Println("PID file created")
 
 	// preparing rule set from rules file
@@ -122,5 +124,8 @@ func Stop() {
 	if err != nil {
 		errorLogger.Println(err)
 	}
-	syscall.Kill(pid, syscall.SIGTERM)
+	err = syscall.Kill(pid, syscall.SIGTERM)
+	if err != nil {
+		errorLogger.Println(err)
+	}
 }