renamed clientId to userAgentHeader in connect args

examples/repl

@@ -15,7 +15,7 @@ async function initClient({ host, endpointId, token }) {
         host,
         path: `/sql/2.0/warehouses/${endpointId}`,
         token,
-        clientId: 'REPL',
+        userAgentHeader: 'REPL',
     });
 }
 

lib/DBSQLClient.ts

@@ -111,7 +111,7 @@ export default class DBSQLClient extends EventEmitter implements IDBSQLClient, I
       socketTimeout: options.socketTimeout,
       proxy: options.proxy,
       headers: {
-        'User-Agent': buildUserAgentString(),
+        'User-Agent': buildUserAgentString(options.userAgentHeader),
       },
     };
   }

lib/contracts/IDBSQLClient.ts

@@ -30,7 +30,7 @@ export type ConnectionOptions = {
   host: string;
   port?: number;
   path: string;
-  clientId?: string;
+  userAgentHeader?: string;
   socketTimeout?: number;
   proxy?: ProxyOptions;
 } & AuthOptions;

lib/utils/buildUserAgentString.ts

@@ -11,7 +11,7 @@ function getOperatingSystemVersion(): string {
   return `${os.type()} ${os.release()}`;
 }
 
-export default function buildUserAgentString(): string {
-  const extra = [getNodeVersion(), getOperatingSystemVersion()].filter(Boolean);
+export default function buildUserAgentString(userAgentHeader?: string): string {
+  const extra = [userAgentHeader, getNodeVersion(), getOperatingSystemVersion()].filter(Boolean);
   return `${productName}/${packageVersion} (${extra.join('; ')})`;
 }

tests/unit/utils/utils.test.ts

@@ -17,16 +17,16 @@ const progressUpdateResponseStub: TProgressUpdateResp = {
 describe('buildUserAgentString', () => {
   // It should follow https://www.rfc-editor.org/rfc/rfc7231#section-5.5.3 and
   // https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/User-Agent
-  //   UserAgent ::= <ProductName> '/' <ProductVersion> '(' <Comment> ')'
-  // where:
-  //   <ProductName> is "NodejsDatabricksSqlConnector"
-  //   <ProductVersion> is three period-separated digits (optionally with a suffix)
-  //   <Comment> is "Node.js <NodeJsVersion>; <OSPlatform> <OSVersion>"
   //
-  // Example:
-  // - NodejsDatabricksSqlConnector/0.1.8-beta.1 (Node.js 16.13.1; Darwin 21.5.0)
+  // UserAgent ::= <ProductName> '/' <ProductVersion> '(' <Comment> ')'
+  // ProductName ::= 'NodejsDatabricksSqlConnector'
+  // <Comment> ::= [ <userAgentHeader> ';' ] 'Node.js' <NodeJsVersion> ';' <OSPlatform> <OSVersion>
+  //
+  // Examples:
+  // - with <userAgentHeader> provided: NodejsDatabricksSqlConnector/0.1.8-beta.1 (Client ID; Node.js 16.13.1; Darwin 21.5.0)
+  // - without <userAgentHeader> provided: NodejsDatabricksSqlConnector/0.1.8-beta.1 (Node.js 16.13.1; Darwin 21.5.0)
 
-  function checkUserAgentString(ua: string) {
+  function checkUserAgentString(ua: string, userAgentHeader?: string) {
     // Prefix: 'NodejsDatabricksSqlConnector/'
     // Version: three period-separated digits and optional suffix
     const re =
@@ -39,13 +39,18 @@ describe('buildUserAgentString', () => {
     const parts = comment.split(';').map((s) => s.trim());
     expect(parts.length).to.be.gte(2); // at least Node and OS version should be there
 
-    // First part should start with "Node.js" followed by a version number.
-    expect(parts[0]).to.match(/^Node\.js\s+\d+\.\d+\.\d+/);
-    // Second part should represent the OS platform (a word) and OS version.
-    expect(parts[1]).to.match(/^\w+/);
+    if (userAgentHeader) {
+      expect(comment.trim()).to.satisfy((s: string) => s.startsWith(`${userAgentHeader};`));
+    }
   }
 
-  it('matches pattern', () => {
+  it('matches pattern with userAgentHeader', () => {
+    const userAgentHeader = 'Some Client ID';
+    const ua = buildUserAgentString(userAgentHeader);
+    checkUserAgentString(ua, userAgentHeader);
+  });
+
+  it('matches pattern without userAgentHeader', () => {
     const ua = buildUserAgentString();
     checkUserAgentString(ua);
   });