Skip to content

Commit

Permalink
fix(activities): only search for tmp chat download path when finding …
Browse files Browse the repository at this point in the history
…process to kill
  • Loading branch information
Zibbp committed May 30, 2024
1 parent 1038c62 commit b67d7b8
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion internal/activities/video.go
Original file line number Diff line number Diff line change
Expand Up @@ -707,7 +707,7 @@ func KillTwitchLiveChatDownload(ctx context.Context, input dto.ArchiveVideoInput

// find pid of chat_downloader to kill
// search for channel and unique temporary download path to ensure we do not kill a new instance
cmd := osExec.Command("pgrep", "-f", fmt.Sprintf("chat_downloader https://twitch.tv/%s --output %s", input.Channel.Name, input.Vod.TmpLiveChatDownloadPath))
cmd := osExec.Command("pgrep", "-f", fmt.Sprintf("%s", input.Vod.TmpLiveChatDownloadPath))

Check failure on line 710 in internal/activities/video.go

View workflow job for this annotation

GitHub Actions / lint

S1025: the argument is already a string, there's no need to use fmt.Sprintf (gosimple)
out, err := cmd.Output()
if err != nil {
return temporal.NewApplicationError(err.Error(), "", nil)
Expand Down

0 comments on commit b67d7b8

Please sign in to comment.