opentelemetry-instrumentation-nestjs-core - kills the ability to define interceptors on the handler level

[danshapir]

What version of OpenTelemetry are you using?

0.25

What version of Node are you using?

16

What did you do?

Adding the opentelemetry-instrumentation-nestjs-core creates a scenario where using the
@UseInterceptors(WHAT_EVER) on the handler level (meaning a method), doesn't work! Those were 11 hours of evil debugging.

What did you expect to see?

What did you see instead?

Additional context

[austonpramodh]

I see that metadata assigned with the decorator also gets messed up.

export const isPublicMetaDataKey = "isPublic";
export const Public = (): CustomDecorator<string> => SetMetadata(isPublicMetaDataKey, true);

if I have Public() on a handler, this.reflector.get<boolean>(isPublicMetaDataKey, context.getHandler()); doesn't get the value.

[Update]
#633 is more relevant for this comment.

#633 (comment)

[github-actions]

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 14 days.

[danshapir]

Still not working

[matteoga]

i think #796 should have fixed this and similar issues

[dyladan]

@danshapir can you confirm that this is fixed by #796 ?