fix(url parser): try catch bug, not actually returning from try loop

Tests that threw an error would trigger the error block of the try/catch. Is rewritten now to avoid this but avoiding try/catch probably best in the long run with regards to callbacks.

Author: Jessica Lord

lib/url_parser.js

@@ -47,12 +47,14 @@ module.exports = function(url, options, callback) {
 };
 
 function parseHandler(address, options, callback) {
+  let result, err;
   try {
-    const result = parseConnectionString(address, options);
-    return callback(null, result);
-  } catch (err) {
-    return callback(err);
+    result = parseConnectionString(address, options);
+  } catch (e) {
+    err = e;
   }
+
+  return err ? callback(err, null) : callback(null, result);
 }
 
 function parseConnectionString(url, options) {