Only start network monitoring service after the first call to connect in METDDPClient

Author: tonyxiao

Meteor/METDDPClient.m

@@ -107,7 +107,6 @@ - (instancetype)initWithConnection:(METDDPConnection *)connection account:(METAc
     _networkReachabilityManager = [[METNetworkReachabilityManager alloc] initWithHostName:connection.serverURL.host];
     _networkReachabilityManager.delegate = self;
     _networkReachabilityManager.delegateQueue = _queue;
-    [_networkReachabilityManager startMonitoring];
 
     _keepAliveBackgroundTask = UIBackgroundTaskInvalid;
 
@@ -188,6 +187,7 @@ - (BOOL)isConnected {
 
 - (void)connect {
   @synchronized(self) {
+    [_networkReachabilityManager startMonitoring];
     if (_connectionStatus == METDDPConnectionStatusOffline || _connectionStatus == METDDPConnectionStatusWaiting) {
       if ([_delegate respondsToSelector:@selector(clientWillConnect:)]) {
         [_delegate clientWillConnect:self];

Meteor/METNetworkReachabilityManager.m

@@ -35,6 +35,7 @@ static void METNetworkReachabilityCallback(SCNetworkReachabilityRef target, SCNe
 
 @implementation METNetworkReachabilityManager {
   SCNetworkReachabilityRef _reachabilityRef;
+  BOOL _started;
 }
 
 #pragma mark - Lifecycle
@@ -61,14 +62,15 @@ - (void)dealloc {
 
 - (BOOL)startMonitoring {
   NSAssert(_delegateQueue != nil, @"Delegate queue should be set before calling startMonitoring");
-  
+  if (_started) {
+    return YES;
+  }
   SCNetworkReachabilityContext context = {0, (__bridge void *)(self), NULL, NULL, NULL};
 
   if (SCNetworkReachabilitySetCallback(_reachabilityRef, METNetworkReachabilityCallback, &context)) {
-    return SCNetworkReachabilitySetDispatchQueue(_reachabilityRef, _delegateQueue);
+    _started = SCNetworkReachabilitySetDispatchQueue(_reachabilityRef, _delegateQueue);
   }
-  
-  return NO;
+  return _started;
 }
 
 - (void)stopMonitoring {