test

Author: liustve

aws-distro-opentelemetry-node-autoinstrumentation/src/exporter/otlp/aws/traces/otlp-aws-span-exporter.ts

@@ -12,23 +12,22 @@ let HttpRequest: any;
 let defaultProvider: any;
 let Sha256: any;
 
-const nodeVersionSupported = getNodeVersion() >= 16;
+// Flag to track if dependencies were successfully loaded
+let dependenciesLoaded = false;
 
-if (nodeVersionSupported) {
+// Conditionally load dependencies if Node.js version is >= 16
+if (getNodeVersion() >= 16) {
   try {
-    const { defaultProvider: awsDefaultProvider } = require('@aws-sdk/credential-provider-node');
-    const { Sha256: awsSha256 } = require('@aws-crypto/sha256-js');
-    const { SignatureV4: awsSignatureV4 } = require('@smithy/signature-v4');
-    const { HttpRequest: awsHttpRequest } = require('@smithy/protocol-http');
-    
-    // Assign to module-level variables
-    defaultProvider = awsDefaultProvider;
-    Sha256 = awsSha256;
-    SignatureV4 = awsSignatureV4;
-    HttpRequest = awsHttpRequest;
+    defaultProvider = require('@aws-sdk/credential-provider-node').defaultProvider;
+    Sha256 = require('@aws-crypto/sha256-js').Sha256;
+    SignatureV4 = require('@smithy/signature-v4').SignatureV4;
+    HttpRequest = require('@smithy/protocol-http').HttpRequest;
+    dependenciesLoaded = true;
   } catch (error) {
     diag.error(`Failed to load required AWS dependency for SigV4 Signing: ${error}`);
   }
+} else {
+  diag.error('SigV4 signing requires at least Node major version 16');
 }