From 5235d4cafa0675021b69cad646d93eda811b0603 Mon Sep 17 00:00:00 2001
From: cloudflare-warp-bot <warp-bot@cloudflare.com>
Date: Mon, 26 Mar 2018 22:47:23 +0000
Subject: [PATCH] Release Argo Tunnel Client 2018.3.8

---
 cmd/cloudflared/macos_service.go |  4 ++--
 origin/supervisor.go             | 19 ++++++++-----------
 2 files changed, 10 insertions(+), 13 deletions(-)

diff --git a/cmd/cloudflared/macos_service.go b/cmd/cloudflared/macos_service.go
index 95be8454..9278d1af 100644
--- a/cmd/cloudflared/macos_service.go
+++ b/cmd/cloudflared/macos_service.go
@@ -49,8 +49,8 @@ var launchdTemplate = ServiceTemplate{
 		<string>/tmp/%s.err.log</string>
 		<key>KeepAlive</key>
 		<dict>
-			<key>NetworkState</key>
-			<true/>
+			<key>SuccessfulExit</key>
+			<false/>
 		</dict>
 		<key>ThrottleInterval</key>
 		<integer>20</integer>
diff --git a/origin/supervisor.go b/origin/supervisor.go
index 35c73f63..f9e75bd5 100644
--- a/origin/supervisor.go
+++ b/origin/supervisor.go
@@ -11,8 +11,6 @@ import (
 const (
 	// Waiting time before retrying a failed tunnel connection
 	tunnelRetryDuration = time.Second * 10
-	// Limit on the exponential backoff time period. (2^5 = 32 minutes)
-	tunnelRetryLimit = 5
 	// SRV record resolution TTL
 	resolveTTL = time.Hour
 )
@@ -54,30 +52,31 @@ func (s *Supervisor) Run(ctx context.Context, connectedSignal chan struct{}) err
 	if err := s.initialize(ctx, connectedSignal); err != nil {
 		return err
 	}
-	tunnelsActive := s.config.HAConnections
-	tunnelsWaiting := []int{}
+	var tunnelsWaiting []int
 	backoff := BackoffHandler{MaxRetries: s.config.Retries, BaseTime: tunnelRetryDuration, RetryForever: true}
 	var backoffTimer <-chan time.Time
-	for tunnelsActive > 0 {
+
+	for {
 		select {
 		// Context cancelled
 		case <-ctx.Done():
-			for tunnelsActive > 0 {
+			for len(s.tunnelErrors) > 0 {
 				<-s.tunnelErrors
-				tunnelsActive--
 			}
+
 			return nil
 		// startTunnel returned with error
 		// (note that this may also be caused by context cancellation)
 		case tunnelError := <-s.tunnelErrors:
-			tunnelsActive--
 			if tunnelError.err != nil {
 				Log.WithError(tunnelError.err).Warn("Tunnel disconnected due to error")
 				tunnelsWaiting = append(tunnelsWaiting, tunnelError.index)
 				s.waitForNextTunnel(tunnelError.index)
+
 				if backoffTimer == nil {
 					backoffTimer = backoff.BackoffTimer()
 				}
+
 				// If the error is a dial error, the problem is likely to be network related
 				// try another addr before refreshing since we are likely to get back the
 				// same IPs in the same order. Same problem with duplicate connection error.
@@ -93,7 +92,6 @@ func (s *Supervisor) Run(ctx context.Context, connectedSignal chan struct{}) err
 			for _, index := range tunnelsWaiting {
 				go s.startTunnel(ctx, index, s.newConnectedTunnelSignal(index))
 			}
-			tunnelsActive += len(tunnelsWaiting)
 			tunnelsWaiting = nil
 		// Tunnel successfully connected
 		case <-s.nextConnectedSignal:
@@ -113,7 +111,6 @@ func (s *Supervisor) Run(ctx context.Context, connectedSignal chan struct{}) err
 			}
 		}
 	}
-	return fmt.Errorf("All tunnels terminated")
 }
 
 func (s *Supervisor) initialize(ctx context.Context, connectedSignal chan struct{}) error {
@@ -135,7 +132,7 @@ func (s *Supervisor) initialize(ctx context.Context, connectedSignal chan struct
 	case <-ctx.Done():
 		<-s.tunnelErrors
 		// Error can't be nil. A nil error signals that initialization succeed
-		return fmt.Errorf("Context was canceled")
+		return fmt.Errorf("context was canceled")
 	case tunnelError := <-s.tunnelErrors:
 		return tunnelError.err
 	case <-connectedSignal: