cleared up logic

Author: aditi-khare-mongoDB

Diff for: src/cmap/handshake/client_metadata.ts

@@ -173,26 +173,17 @@ export async function addContainerMetadata(originalMetadata: ClientMetadata) {
     }
     const isKubernetes = process.env.KUBERNETES_SERVICE_HOST ? true : false;
 
-    if (isDocker || isKubernetes) {
-      if (isDocker) {
-        containerMetadata['runtime'] = 'docker';
-      }
-      if (isKubernetes) {
-        containerMetadata['orchestrator'] = 'kubernetes';
-      }
-    }
+    if (isDocker) containerMetadata['runtime'] = 'docker';
+    if (isKubernetes) containerMetadata['orchestrator'] = 'kubernetes';
+
     return containerMetadata;
   }
 
   const containerMetadata = await getContainerMetadata();
   if (Object.keys(containerMetadata).length === 0) return originalMetadata;
 
   const extendedMetadata = new LimitedSizeDocument(512);
-
-  let envMetadata = { container: containerMetadata };
-  if ('env' in originalMetadata) {
-    envMetadata = { ...originalMetadata['env'], ...envMetadata };
-  }
+  const envMetadata = { ...originalMetadata?.env, container: containerMetadata };
 
   for (const [key, val] of Object.entries(originalMetadata)) {
     if (key !== 'env') {