fixes

Author: ChristopherHX

main.go

@@ -222,9 +222,7 @@ func (svc *RunRunnerSvc) Start(s service.Service) error {
 		runner.Terminal = terminal
 	}
 
-	//nolint:gosec
 	ctx, cancel := context.WithCancel(context.Background())
-	//nolint:gosec
 	listenerctx, cancelListener := context.WithCancel(context.Background())
 	svc.stop = func() {
 		cancelListener()

protocol/logger/job_logger.go

@@ -254,7 +254,7 @@ type BufferedLiveLogger struct {
 	data atomic.Pointer[internalBufferedLiveLoggerData]
 }
 
-func (logger *BufferedLiveLogger) sendLogs(logchan chan *protocol.TimelineRecordFeedLinesWrapper, logdrain chan struct{}, logfinished chan struct{}) {
+func (logger *BufferedLiveLogger) sendLogs(logchan chan *protocol.TimelineRecordFeedLinesWrapper, logdrain, logfinished chan struct{}) {
 	defer func() {
 		if r := recover(); r != nil {
 			log.Printf("panic recovered: %v\n%s", r, debug.Stack())

protocol/logger/job_logger_test.go

@@ -6,6 +6,7 @@ import (
 	"time"
 
 	"github.com/ChristopherHX/github-act-runner/protocol"
+
 	"github.com/stretchr/testify/require"
 )
 
@@ -60,13 +61,14 @@ func getTestLiveLogger(t *testing.T) LiveLogger {
 }
 
 func TestBufferedLiveLoggerDrain(t *testing.T) {
+	t.Parallel()
 	bufferedLogger := &BufferedLiveLogger{
 		LiveLogger: getTestLiveLogger(t),
 	}
 
-	var logchan chan *protocol.TimelineRecordFeedLinesWrapper = make(chan *protocol.TimelineRecordFeedLinesWrapper)
-	var logdrain chan struct{} = make(chan struct{})
-	var logfinished chan struct{} = make(chan struct{})
+	logchan := make(chan *protocol.TimelineRecordFeedLinesWrapper)
+	logdrain := make(chan struct{})
+	logfinished := make(chan struct{})
 
 	t.Run("forwardLogs", func(t *testing.T) {
 		t.Parallel()
@@ -89,6 +91,7 @@ func TestBufferedLiveLoggerDrain(t *testing.T) {
 }
 
 func TestBufferedLiveLogger(t *testing.T) {
+	t.Parallel()
 	bufferedLogger := &BufferedLiveLogger{
 		LiveLogger: getTestLiveLogger(t),
 	}