Merge pull request #197 from charankamarapu/mockLib

charankamarapu · web-flow · commit 5c345c116018 · 2023-09-26T12:41:49.000+05:30
fix: kill process with SIGTERM
diff --git a/keploy/mock.go b/keploy/mock.go
@@ -152,10 +152,12 @@ func KillProcessOnPort() {
 			forceKillProcessByPID(pid)
 		}
 	}
+	// waiting for exit logs to print
+	time.Sleep(time.Second)
 }
 
 func forceKillProcessByPID(pid string) {
-	cmd := exec.Command("sudo", "kill", "-9", pid)
+	cmd := exec.Command("sudo", "kill", "-s", "SIGTERM", pid)
 	if err := cmd.Run(); err != nil {
 		logger.Error(fmt.Sprintf("Failed to kill process with PID %s:", pid), zap.Error(err))
 	}

Original file line number	Diff line number	Diff line change
`@@ -152,10 +152,12 @@ func KillProcessOnPort() {`
`152`	`152`	`forceKillProcessByPID(pid)`
`153`	`153`	`}`
`154`	`154`	`}`
	`155`	`+ // waiting for exit logs to print`
	`156`	`+ time.Sleep(time.Second)`
`155`	`157`	`}`
`156`	`158`
`157`	`159`	`func forceKillProcessByPID(pid string) {`
`158`		`- cmd := exec.Command("sudo", "kill", "-9", pid)`
	`160`	`+ cmd := exec.Command("sudo", "kill", "-s", "SIGTERM", pid)`
`159`	`161`	`if err := cmd.Run(); err != nil {`
`160`	`162`	`logger.Error(fmt.Sprintf("Failed to kill process with PID %s:", pid), zap.Error(err))`
`161`	`163`	`}`