From 23e1b1db3ac0cdf42653db0df4d126d79dcb8a43 Mon Sep 17 00:00:00 2001
From: Michael Avila <davidmichaelavila@gmail.com>
Date: Mon, 6 May 2019 16:16:49 -0700
Subject: [PATCH] Use conditional for running provider online

License: MIT
Signed-off-by: Michael Avila <davidmichaelavila@gmail.com>
---
 core/node/provider.go | 37 +++++++++++++++++++------------------
 1 file changed, 19 insertions(+), 18 deletions(-)

diff --git a/core/node/provider.go b/core/node/provider.go
index dfea3b833..43540745b 100644
--- a/core/node/provider.go
+++ b/core/node/provider.go
@@ -37,23 +37,24 @@ func SimpleReprovider(reproviderInterval time.Duration) interface{} {
 }
 
 // SimpleProviderSys creates new provider system
-func SimpleProviderSys(lc fx.Lifecycle, p provider.Provider, r provider.Reprovider) provider.System {
-	sys := provider.NewSystem(p, r)
-	lc.Append(fx.Hook{
-		OnStart: func(ctx context.Context) error {
-			sys.Run()
-			return nil
-		},
-		OnStop: func(ctx context.Context) error {
-			return sys.Close()
-		},
-	})
-	return sys
-}
+func SimpleProviderSys(isOnline bool) interface{} {
+	return func(lc fx.Lifecycle, p provider.Provider, r provider.Reprovider) provider.System {
+		sys := provider.NewSystem(p, r)
 
-// SimpleOfflineProviderSys creates a new offline provider system
-func SimpleOfflineProviderSys(p provider.Provider, r provider.Reprovider) provider.System {
-	return provider.NewSystem(p, r)
+		if isOnline {
+			lc.Append(fx.Hook{
+				OnStart: func(ctx context.Context) error {
+					sys.Run()
+					return nil
+				},
+				OnStop: func(ctx context.Context) error {
+					return sys.Close()
+				},
+			})
+		}
+
+		return sys
+	}
 }
 
 // ONLINE/OFFLINE
@@ -66,7 +67,7 @@ func OnlineProviders(useStrategicProviding bool, reprovideStrategy string, repro
 
 	return fx.Options(
 		SimpleProviders(reprovideStrategy, reprovideInterval),
-		fx.Provide(SimpleProviderSys),
+		fx.Provide(SimpleProviderSys(true)),
 	)
 }
 
@@ -78,7 +79,7 @@ func OfflineProviders(useStrategicProviding bool, reprovideStrategy string, repr
 
 	return fx.Options(
 		SimpleProviders(reprovideStrategy, reprovideInterval),
-		fx.Provide(SimpleOfflineProviderSys),
+		fx.Provide(SimpleProviderSys(false)),
 	)
 }