fix: data race for get jobs map (#558)

go-co-op · Sep 4, 2023 · fd26595 · fd26595
1 parent b339f73
commit fd26595
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/scheduler.go b/scheduler.go
@@ -142,7 +142,13 @@ func (s *Scheduler) Jobs() []*Job {
 
 // JobsMap returns a map of job uuid to job
 func (s *Scheduler) JobsMap() map[uuid.UUID]*Job {
-	return s.jobsMap()
+	s.jobsMutex.RLock()
+	defer s.jobsMutex.RUnlock()
+	jobs := make(map[uuid.UUID]*Job, len(s.jobs))
+	for id, job := range s.jobs {
+		jobs[id] = job
+	}
+	return jobs
 }
 
 func (s *Scheduler) jobsMap() map[uuid.UUID]*Job {