appsignal-nodejs

appsignal-nodejs copied to clipboard

Add OpenTelemetry RabbitMQ support

15

This patch adds support for `@opentelemetry/instrumentation-amqplib` by including it in the default instrumentation list. It comes with a rest case named `express-rabbitmq`, which is based on the already existing `express-redis`....

jeffkreeftmeijer

Bump express, @nestjs/core, @nestjs/platform-express and @nestjs/testing in /test/nestjs/app

2

Bumps [express](https://github.com/expressjs/express) to 4.19.2 and updates ancestor dependencies [express](https://github.com/expressjs/express), [@nestjs/core](https://github.com/nestjs/nest/tree/HEAD/packages/core), [@nestjs/platform-express](https://github.com/nestjs/nest/tree/HEAD/packages/platform-express) and [@nestjs/testing](https://github.com/nestjs/nest/tree/HEAD/packages/testing). These dependencies need to be updated together. Updates `express` from 4.18.2 to 4.19.2 Release notes Sourced from...

dependabot[bot]

Remove the need for an Express error handler

1

The OpenTelemetry Express instrumentation now automatically records exceptions without the need of an error handler. See: https://github.com/open-telemetry/opentelemetry-js-contrib/pull/1657 ### ToDo - [ ] Update the integration dependency - [ ] Update...

luismiramirez

We mostly test the AppSignal for Node.js integration with npm, which lets you do _whatever_ in terms of dependency use. When `@appsignal/nodejs` attempts to `require("@opentelemetry/sdk-metrics")`, which is not a declared...

unflxw

Research and fix issues with Next.js 14

11

Intercom: https://app.intercom.com/a/inbox/yzor8gyw/inbox/admin/5246522/conversation/16410700308049?view=List Customer is using Next.js 14.1.3. Performance samples are reported as `UNKNOWN /some-path` -- indicating that `http.method` is not provided in the span that is treated as the root...

unflxw

The diagnose report, when executed via `npx @appsignal/cli@latest diagnose` (which is the currently recommended way to execute it) checks for the presence of a `node_modules` in the current folder, or...

unflxw

Our existing HTTP instrumentation does not instrument calls through `fetch`, as it is a completely different beast internally. There's an OpenTelemetry instrumentation at: https://github.com/open-telemetry/opentelemetry-js-contrib/tree/main/plugins/node/instrumentation-undici

unflxw

Implement support for Bull instrumentation

1

We know [some of our users use it](https://app.intercom.com/a/apps/yzor8gyw/inbox/inbox/search/conversations/16410700133028?q=bull&teamAssigneeIdentifier=all&teammateAssigneeIdentifier=all) and have written custom instrumentation for it, and a [non-contrib OpenTelemetry instrumentation](https://github.com/jenniferplusplus/opentelemetry-instrumentation-bullmq) for it exists. We could add support for it, and...

unflxw

Fastify instrumentation may not report errors?

3

We saw an issue during a call where a customer ([Intercom link](https://app.intercom.com/a/inbox/yzor8gyw/inbox/shared/all/conversation/16410700278096#part_id=comment-16410700278096-21909547734)) did not see errors reported from their Fastify application's error handlers automatically. (This customer was affected by #974,...

unflxw

AppSignal shows a scary-looking error on Windows

2

AppSignal should print an error message when on Windows, warning the user that AppSignal is not running -- but it should not look like it crashed. The error seen here...

unflxw