addressed review comments

Azure · Jan 15, 2024 · e54e690 · e54e690
1 parent 6793bd7
commit e54e690
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/src/ServerClient/ADXClient.ts b/src/ServerClient/ADXClient.ts
@@ -1,5 +1,4 @@
 import { ADXResponse, RawADXResponse } from "./ADXResponse";
-const packageJson = require('../../package.json');
 
 type ADXTokenProvider = () => Promise<string>;
 
@@ -72,8 +71,9 @@ export class ADXClient {
         Authorization: `Bearer ${token}`,
         "x-ms-client-request-id": `KTrender;${self.crypto.randomUUID()}`,
         "x-ms-app": "KustoTrender",
-        "x-ms-client-version": packageJson.version,
-        "x-ms-user" : "KustoTrender"
+        "x-ms-client-version": process.env.PACKAGE_VERSION,
+        "x-ms-user" : "KustoTrender",
+        "User-Agent" : navigator.userAgent
       },
       body: JSON.stringify(body),
     });

diff --git a/webpack.common.js b/webpack.common.js
@@ -1,5 +1,7 @@
 const path = require('path');
 const MiniCssExtractPlugin = require("mini-css-extract-plugin");
+const packageJson = require('./package.json');
+const webpack = require('webpack');
 
 module.exports = {
   entry: './src/KustoTrender.ts',
@@ -48,4 +50,9 @@ module.exports = {
     publicPath: '/dist/',
     libraryTarget: 'umd'
   },
+  plugins: [
+    new webpack.DefinePlugin({
+      'process.env.PACKAGE_VERSION': JSON.stringify(packageJson.version)
+    })
+  ]
 };