docs: added comment for "shouldUseLambdaExtension"

kirrg001 · kirrg001 · commit fe8a4fb22a4b · 2025-03-05T11:55:05.000+01:00
- extracted from #1315
diff --git a/packages/aws-lambda/src/wrapper.js b/packages/aws-lambda/src/wrapper.js
@@ -343,6 +343,9 @@ function getRemainingTimeInMillis(context) {
   }
 }
 
+// NOTE: This function only "guesses" whether the Lambda extension should be used or not.
+// TODO: Figure out how we can reliably determine whether the Lambda extension should be
+//       used or not e.g. by checking the lambda handler name if that is possible.
 function shouldUseLambdaExtension() {
   if (process.env.INSTANA_DISABLE_LAMBDA_EXTENSION) {
     logger.info('INSTANA_DISABLE_LAMBDA_EXTENSION is set, not using the Lambda extension.');

Original file line number	Diff line number	Diff line change
`@@ -343,6 +343,9 @@ function getRemainingTimeInMillis(context) {`
`343`	`343`	`}`
`344`	`344`	`}`
`345`	`345`
	`346`	`+// NOTE: This function only "guesses" whether the Lambda extension should be used or not.`
	`347`	`+// TODO: Figure out how we can reliably determine whether the Lambda extension should be`
	`348`	`+// used or not e.g. by checking the lambda handler name if that is possible.`
`346`	`349`	`function shouldUseLambdaExtension() {`
`347`	`350`	`if (process.env.INSTANA_DISABLE_LAMBDA_EXTENSION) {`
`348`	`351`	`logger.info('INSTANA_DISABLE_LAMBDA_EXTENSION is set, not using the Lambda extension.');`