upstream/webhookd
1
0
Fork 0
mirror of https://github.com/ncarlier/webhookd.git synced 2024-09-20 00:45:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: bufio.Scanner: token too long

Browse Source 
close #98
This commit is contained in:
Nicolas Carlier 2024-08-18 07:17:28 +00:00
parent 0c41f76f53
commit 8f8e69faae
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
pkg/hook/job.go
View File

@ -242,9 +242,17 @@ func (job *Job) Run() error {
// Write script output to log file and the work message channel
go func(reader io.Reader) {
scanner := bufio.NewScanner(reader)
for scanner.Scan() {
line := scanner.Text()
r := bufio.NewReader(reader)
for {
line, err := r.ReadString('\n')
if err != nil {
if err == io.EOF {
break
}
slog.Error("error while reading hook std[out/err]", "hook", job.name, "id", job.id, "err", err)
break
}
line, _ = strings.CutSuffix(line, "\r")
// writing to the work channel
if !job.IsTerminated() {
job.MessageChan <- []byte(line)
@ -266,9 +274,6 @@ func (job *Job) Run() error {
break
}
}
if err := scanner.Err(); err != nil {
slog.Error("hook is unable to read script stdout", "hook", job.name, "id", job.id, "err", err)
}
wg.Done()
}(cmdReader)