Improve url handling

Author: vikrantpuppala

lib/connection/connections/HttpConnection.ts

@@ -98,7 +98,7 @@ export default class HttpConnection implements IConnectionProvider {
 
       this.connection = new ThriftHttpConnection(
         {
-          url: `${options.https ? 'https' : 'http'}://${options.host.replace(/\/+$/, '')}:${options.port}${options.path ? ('/' + options.path.replace(/^\/+|\/+$/g, '')) : '/'}`,
+          url: `${options.https ? 'https' : 'http'}://${options.host.replace(/\/$/, '')}:${options.port}${options.path ? ('/' + options.path.replace(/\/$/, '')) : '/'}`,
           transport: thrift.TBufferedTransport,
           protocol: thrift.TBinaryProtocol,
           getRetryPolicy: () => this.getRetryPolicy(),

tests/unit/connection/connections/HttpConnection.test.ts

@@ -150,10 +150,6 @@ describe('HttpConnection.connect', () => {
         input: { host: 'xyz.com/', path: 'sql/v1' },
         expected: 'https://xyz.com:443/sql/v1'
       },
-      {
-        input: { host: 'xyz.com//', path: '//sql/v1//' },
-        expected: 'https://xyz.com:443/sql/v1'
-      },
       {
         input: { host: 'xyz.com', path: undefined },
         expected: 'https://xyz.com:443/'