fix: experimental metrics middleware should use the grpc message cons…

…tructor name in emitted metrics instead of MiddlewareMessage (#993)

* fix: experimental metrics middleware should use the grpc message constructor name in emitted metrics instead of MiddlewareMessage

* add null and undefined check

* fix lint errors

* move null check to middleware message instead

* remove console log and run linter

packages/client-sdk-nodejs/src/config/middleware/impl/experimental-metrics-middleware.ts

@@ -108,7 +108,7 @@ export abstract class ExperimentalMetricsMiddlewareRequestHandler
 
   onRequestBody(request: MiddlewareMessage): Promise<MiddlewareMessage> {
     this.requestSize = request.messageLength();
-    this.requestType = request.constructor.name;
+    this.requestType = request._grpcMessage.constructor.name;
     this.requestBodyTime = new Date().getTime();
     return Promise.resolve(request);
   }

packages/client-sdk-nodejs/src/config/middleware/middleware.ts

@@ -29,7 +29,10 @@ export class MiddlewareMessage {
   }
 
   messageLength(): number {
-    return this._grpcMessage.serializeBinary().length;
+    if (this._grpcMessage !== null && this._grpcMessage !== undefined) {
+      return this._grpcMessage.serializeBinary().length;
+    }
+    return 0;
   }
 }
 

packages/client-sdk-nodejs/src/index.ts

@@ -200,6 +200,7 @@ export {
 
 export {ExperimentalRequestLoggingMiddleware} from './config/middleware/experimental-request-logging-middleware';
 export {ExperimentalMetricsCsvMiddleware} from './config/middleware/experimental-metrics-csv-middleware';
+export {ExperimentalMetricsLoggingMiddleware} from './config/middleware/experimental-metrics-logging-middleware';
 export {ExampleAsyncMiddleware} from './config/middleware/example-async-middleware';
 
 export {