Merge pull request #884 from bhandras/notification-no-busy-wait

notifications: do not hog the CPU if there's no L402 token yet

Author: bhandras

notifications/manager.go

@@ -149,14 +149,17 @@ func (m *Manager) Run(ctx context.Context) error {
 		waitTime time.Duration
 		backoff  time.Duration
 		attempts int
+		timer    = time.NewTimer(0)
 	)
 
 	// Start the notification runloop.
 	for {
 		// Increase the wait time for the next iteration.
 		backoff = waitTime + time.Duration(attempts)*time.Second
 		waitTime = 0
-		timer := time.NewTimer(backoff)
+
+		// Reset the timer with the new backoff time.
+		timer.Reset(backoff)
 
 		// Return if the context has been canceled.
 		select {
@@ -179,6 +182,10 @@ func (m *Manager) Run(ctx context.Context) error {
 					log.Errorf("Error getting L402 from "+
 						"the store: %v", err)
 				}
+
+				// Use a default of 1 second wait time to avoid
+				// hogging the CPU.
+				waitTime = time.Second
 				continue
 			}