Merge pull request #313 from SamuelSarle/master

Fix resource leak from time.NewTicker
mailgun · Nov 6, 2023 · d9eae83 · d9eae83
2 parents b3130dc + ba99848
commit d9eae83
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/events.go b/events.go
@@ -277,12 +277,12 @@ func (ep *EventPoller) Poll(ctx context.Context, events *[]Event) bool {
  ep.it.Paging.Next = currentPage
 
  // Sleep the rest of our duration
- tick := time.NewTicker(ep.opts.PollInterval)
+ timer := time.NewTimer(ep.opts.PollInterval)
  select {
  case <-ctx.Done():
+ timer.Stop()
  return false
- case <-tick.C:
- tick.Stop()
+ case <-timer.C:
  }
  }