Fixed issue where connecting could take very long and cancellation would not work (#78)

* Fixed issue where connecting could take very long and cancellation would not work

* Addressing feedback

* Remove warning suppression

src/Microsoft.SqlTools.ServiceLayer/Connection/ConnectionService.cs

@@ -264,10 +264,7 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection
             };
 
             // invoke callback notifications
-            foreach (var activity in this.onConnectionActivities)
+            invokeOnConnectionActivities(connectionInfo);
-            {
-                await activity(connectionInfo);
-            }
 
             // try to get information about the connected SQL Server instance
             try
@@ -461,24 +458,7 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection
 
             try
             {
-                // create a task to connect asyncronously so that other requests are not blocked in the meantime
+                RunConnectRequestHandlerTask(connectParams, requestContext);
-                Task.Run(async () => 
-                {
-                    try
-                    {
-                        // open connection based on request details
-                        ConnectionCompleteParams result = await ConnectionService.Instance.Connect(connectParams);
-                        await ServiceHost.SendEvent(ConnectionCompleteNotification.Type, result);
-                    }
-                    catch (Exception ex)
-                    {
-                        ConnectionCompleteParams result = new ConnectionCompleteParams()
-                        {
-                            Messages = ex.ToString()
-                        };
-                        await ServiceHost.SendEvent(ConnectionCompleteNotification.Type, result);
-                    }
-                });
                 await requestContext.SendResult(true);
             }
             catch
@@ -487,6 +467,28 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection
             }
         }
 
+        private void RunConnectRequestHandlerTask(ConnectParams connectParams, RequestContext<bool> requestContext)
+        {
+            // create a task to connect asynchronously so that other requests are not blocked in the meantime
+            Task.Run(async () => 
+            {
+                try
+                {
+                    // open connection based on request details
+                    ConnectionCompleteParams result = await ConnectionService.Instance.Connect(connectParams);
+                    await ServiceHost.SendEvent(ConnectionCompleteNotification.Type, result);
+                }
+                catch (Exception ex)
+                {
+                    ConnectionCompleteParams result = new ConnectionCompleteParams()
+                    {
+                        Messages = ex.ToString()
+                    };
+                    await ServiceHost.SendEvent(ConnectionCompleteNotification.Type, result);
+                }
+            });
+        }
+
         /// <summary>
         /// Handle cancel connect requests
         /// </summary>
@@ -723,5 +725,14 @@ namespace Microsoft.SqlTools.ServiceLayer.Connection
                 }
             }
         }
+
+        private void invokeOnConnectionActivities(ConnectionInfo connectionInfo)
+        {
+            foreach (var activity in this.onConnectionActivities)
+            {
+                // not awaiting here to allow handlers to run in the background
+                activity(connectionInfo);
+            }
+        }
     }
 }