opentelemetry-instrumentation-aws-lambda silently fails to instrument files with the extension.cjs

[Ankcorn]

What version of OpenTelemetry are you using?

Will be upgrading to the latest
"@opentelemetry/api": "^1.3.0",
"@opentelemetry/exporter-trace-otlp-http": "^0.34.0",
"@opentelemetry/instrumentation": "^0.34.0",
"@opentelemetry/instrumentation-aws-lambda": "^0.34.0",
"@opentelemetry/instrumentation-aws-sdk": "^0.33.0",
"@opentelemetry/instrumentation-http": "^0.34.0",
"@opentelemetry/sdk-trace-base": "^1.8.0",
"@opentelemetry/sdk-trace-node": "^1.8.0",

What version of Node are you using?

v16.x

What did you do?

https://github.com/baselime/lambda-node-opentelemetry-serverless-stack-v2
Here is a repo reproducing the issue

What did you expect to see?

Traces sent

What did you see instead?

lambda handler is not instrumented

Additional context

opentelemetry-js-contrib/plugins/node/opentelemetry-instrumentation-aws-lambda/src/instrumentation.ts

Line 103 in 9605528

filename += '.js';

issue is this line of code does not handle all possible options supported by lambda node runtime

[dyladan]

@willarmiros can you please look into this and the linked PR?

[Ankcorn]

Thanks for getting this published!

I just saw this open-telemetry/opentelemetry-js#3698 pr was closed!

I realize that makes my fix a little half baked. sorry .mjs files...

Should I make a new issue to track that @dyladan

[dyladan]

Yes