Issue with button handling in production

[victorsmits]

What is your question?

For my bot I am using some buttons and in local I encounter no issue at all but in production it seams that the handler broke my app with this error

node:events:498
      throw er; // Unhandled 'error' event
      ^
TypeError: Cannot read properties of null (reading 'isHeadersSent')
    at SentryGlobalFilter.handleUnknownError (/app/node_modules/@nestjs/core/exceptions/base-exception-filter.js:45:29)
    at SentryGlobalFilter.catch (/app/node_modules/@nestjs/core/exceptions/base-exception-filter.js:17:25)
    at SentryGlobalFilter.catch (/app/node_modules/@sentry/nestjs/build/cjs/setup.js:93:23)
    at ExternalExceptionsHandler.invokeCustomFilters (/app/node_modules/@nestjs/core/exceptions/external-exceptions-handler.js:31:32)
    at ExternalExceptionsHandler.next (/app/node_modules/@nestjs/core/exceptions/external-exceptions-handler.js:14:29)
    at MessageComponentDiscovery.contextCallback (/app/node_modules/@nestjs/core/helpers/external-proxy.js:14:42)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Emitted 'error' event on Client instance at:
    at emitUnhandledRejectionOrErr (node:events:403:10)
    at process.processTicksAndRejections (node:internal/process/task_queues:84:21)
Node.js v22.2.0
npm notice
npm notice New major version of npm available! 10.7.0 -> 11.0.0
npm notice Changelog: https://github.com/npm/cli/releases/tag/v11.0.0
npm notice To update run: npm install -g [email protected]
npm notice
> [email protected] start:prod
> node dist/main

I do not understand why its happening only in production

[SocketSomeone]

Hi, what is isHeadersSent method? Can you create repro?

[victorsmits]

The global filter come from sentry.io I did not build it.

[SocketSomeone]

The global filter come from sentry.io I did not build it.

So, it can be a reason of problem. As I can see it tries to access isHeadersSent method, but Necord doest have HTTP API or something else. So, i think you need extend sentry global filter to correctly process exceptions.

I found that SentryGlobalFilter is inherit from BaseException filter which using this method when error is unknown for sentry global filter

I think you can do something like

@Catch()
class CustomSentryGlobalFilter extends SentryGlobalFilter {
  public constructor(applicationRef?: HttpServer) {
    super(applicationRef);
    this.__SENTRY_INTERNAL__ = true;
    this._logger = new Logger('ExceptionsHandler');
  }

  /**
   * Catches exceptions and reports them to Sentry unless they are expected errors.
   */
  public catch(exception: unknown, host: ArgumentsHost): void {
    if (host.getType<'necord'>() === 'necord') {
      if (exception instanceof Error) {
        this._logger.error(exception.message, exception.stack);
      }

      captureException(exception);
      throw exception;
    }
    
    return super.catch(exception, host);
  }
}