Fix cpu leak from unclosed tickers (#21)

* fix ticker-related cpu leak * Update CHANGELOG.md
Reidmcc · Feb 23, 2019 · 4f2383d · 4f2383d
1 parent 970ce57
commit 4f2383d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,6 +4,8 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+### Unreleased
+- Fixed a cpu leak from unclosed tickers in stream routines
 
 ## v 0.3.0-alpha
 

diff --git a/modules/dexWatcher.go b/modules/dexWatcher.go
@@ -277,10 +277,8 @@ func (w *DexWatcher) streamOld(
 	defer resp.Body.Close()
 
 	reader := bufio.NewReader(resp.Body)
-	ticker := time.NewTicker(10 * time.Millisecond)
-
 	for {
-		<-ticker.C
+		time.Sleep(10 * time.Millisecond)
 	Events:
 		for {
 			var buffer bytes.Buffer