Fix crash in NetworkService due to components other than NetworkServiceManager binding to the service without the proper initialization information. This initialization was moved to another method named bootstrapService which is only called from NetworkServiceManager after the NetworkService has been properly connected.

graphenej/src/main/java/cy/agorise/graphenej/api/android/NetworkService.java

@@ -81,7 +81,7 @@ public class NetworkService extends Service {
     public static final int NORMAL_CLOSURE_STATUS = 1000;
 
     // Time to wait before retrying a connection attempt
-    private final int DEFAULT_RETRY_DELAY = 500;
+    private static final int DEFAULT_RETRY_DELAY = 500;
 
     // Default connection delay when using the node latency verification strategy. This initial
     // delay is required in order ot make sure we have a fair selection of node latencies from
@@ -284,7 +284,17 @@ public class NetworkService extends Service {
     @Nullable
     @Override
     public IBinder onBind(Intent intent) {
-        Bundle extras = intent.getExtras();
+        return mBinder;
+    }
+
+    /**
+     * Initialize information and try to connect to a node accordingly. This methods were moved
+     * from onBind to avoid crashes due to components other than {@link NetworkServiceManager}
+     * binding to the service without submitting the proper information.
+     *
+     * @param extras    Bundle that contains all required information for a proper initialization
+     */
+    public void bootstrapService(Bundle extras) {
         // Retrieving credentials and requested API data from the shared preferences
         mUsername = extras.getString(NetworkService.KEY_USERNAME, "");
         mPassword = extras.getString(NetworkService.KEY_PASSWORD, "");
@@ -327,7 +337,6 @@ public class NetworkService extends Service {
                 mHandler.postDelayed(mConnectAttempt, DEFAULT_INITIAL_DELAY);
             }
         }
-        return mBinder;
     }
 
     /**

graphenej/src/main/java/cy/agorise/graphenej/api/android/NetworkServiceManager.java

@@ -25,14 +25,14 @@ import cy.agorise.graphenej.stats.ExponentialMovingAverage;
  */
 
 public class NetworkServiceManager implements Application.ActivityLifecycleCallbacks  {
-    private final static String TAG = "NetworkServiceManager";
+    private final String TAG = this.getClass().getName();
     /**
      * Constant used to specify how long will the app wait for another activity to go through its starting life
      * cycle events before running the teardownConnectionTask task.
      *
      * This is used as a means to detect whether or not the user has left the app.
      */
-    private final int DISCONNECT_DELAY = 1500;
+    private static final int DISCONNECT_DELAY = 1500;
 
     /**
      * Handler instance used to schedule tasks back to the main thread
@@ -72,8 +72,8 @@ public class NetworkServiceManager implements Application.ActivityLifecycleCallb
         }
     };
 
-    public NetworkServiceManager(Context context){
+    private NetworkServiceManager(Context context){
-        mContextReference = new WeakReference<Context>(context);
+        mContextReference = new WeakReference<>(context);
     }
 
     @Override
@@ -89,27 +89,37 @@ public class NetworkServiceManager implements Application.ActivityLifecycleCallb
             // Creating a new Intent that will be used to start the NetworkService
             Context context = mContextReference.get();
             Intent intent = new Intent(context, NetworkService.class);
-
-            // Adding user-provided node URLs
-            StringBuilder stringBuilder = new StringBuilder();
-            Iterator<String> it = mCustomNodeUrls.iterator();
-            while(it.hasNext()){
-                stringBuilder.append(it.next());
-                if(it.hasNext()) stringBuilder.append(",");
-            }
-            String customNodes = stringBuilder.toString();
-
-            // Adding all
-            intent.putExtra(NetworkService.KEY_USERNAME, mUserName)
-                    .putExtra(NetworkService.KEY_PASSWORD, mPassword)
-                    .putExtra(NetworkService.KEY_REQUESTED_APIS, mRequestedApis)
-                    .putExtra(NetworkService.KEY_NODE_URLS, customNodes)
-                    .putExtra(NetworkService.KEY_AUTO_CONNECT, mAutoConnect)
-                    .putExtra(NetworkService.KEY_ENABLE_LATENCY_VERIFIER, mVerifyLatency);
             context.bindService(intent, mServiceConnection, Context.BIND_AUTO_CREATE);
         }
     }
 
+    /**
+     * This method passes all the required information to the NetworkService to properly
+     * initialize itself
+     */
+    private void passRequiredInfoToConfigureService() {
+        // Adding user-provided node URLs
+        StringBuilder stringBuilder = new StringBuilder();
+        Iterator<String> it = mCustomNodeUrls.iterator();
+        while(it.hasNext()){
+            stringBuilder.append(it.next());
+            if(it.hasNext()) stringBuilder.append(",");
+        }
+        String customNodes = stringBuilder.toString();
+
+        Bundle b = new Bundle();
+
+        // Adding all
+        b.putString(NetworkService.KEY_USERNAME, mUserName);
+        b.putString(NetworkService.KEY_PASSWORD, mPassword);
+        b.putInt(NetworkService.KEY_REQUESTED_APIS, mRequestedApis);
+        b.putString(NetworkService.KEY_NODE_URLS, customNodes);
+        b.putBoolean(NetworkService.KEY_AUTO_CONNECT, mAutoConnect);
+        b.putBoolean(NetworkService.KEY_ENABLE_LATENCY_VERIFIER, mVerifyLatency);
+
+        mService.bootstrapService(b);
+    }
+
     @Override
     public void onActivityPaused(Activity activity) {
         mHandler.postDelayed(mDisconnectRunnable, DISCONNECT_DELAY);
@@ -133,6 +143,8 @@ public class NetworkServiceManager implements Application.ActivityLifecycleCallb
             // We've bound to LocalService, cast the IBinder and get LocalService instance
             NetworkService.LocalBinder binder = (NetworkService.LocalBinder) service;
             mService = binder.getService();
+
+            passRequiredInfoToConfigureService();
         }
 
         @Override