Sentry treats all Injectable's with `use` function as middleware

[asomethings]

Is there an existing issue for this?

• I have checked for existing issues https://github.com/getsentry/sentry-javascript/issues
• I have reviewed the documentation https://docs.sentry.io/
• I am using the latest SDK release https://github.com/getsentry/sentry-javascript/releases

How do you use Sentry?

Sentry Saas (sentry.io)

Which SDK are you using?

@sentry/nestjs

SDK Version

8.27.0

Framework Version

Nest.js 10.4.1

Link to Sentry event

https://sentry.io/issues/5765154861/events/e100e2846c104bb293875be2fcc751c5/?project=4506504369405952

Reproduction Example/SDK Setup

StackBlitz

Create a Injectable class with use method

@Injectable()
export class SomeClass {
  async use(amount: number): Promise<number> {
     return amount - 1
  }
}

Steps to Reproduce

1. Create a @Injectable() class with use method
2. Try to use it from anywhere from the code
3. TypeError: Cannot create proxy with a non-object as target or handler will be thrown

Expected Result

The function should be run normally without errors.

Actual Result

TypeError: Cannot create proxy with a non-object as target or handler will be thrown.

It seems it may apply to functions that sentry tries to proxy it (like transform / canActivate / intercept but not tested)

[lforst]

@nicohrubec we should add guards to ensure that we only instrument when we can reasonably assume that we are actually in a middleware.

[nicohrubec]

Hey @asomethings, thanks for writing in. I just opened a PR with a fix. Should be merged next week.