Francesco Gringl-Novy comments

Results 425 comments of


                                            Francesco Gringl-Novy

Mongoose instrumentation does not work for Mongoose >= 7

We have v8.0.0 out, so you can try it out in the stable version today!

Mongoose instrumentation does not work for Mongoose >= 7

Out of curiosity, did mongoose v7/v8 work in Sentry v7? You did the best thing to move this forward already, which is to leave a comment and show interest upstream...

Mongoose instrumentation does not work for Mongoose >= 7

Great findings! So I guess more stuff to fix upstream 😅 We're putting this on our agenda to try to get these things fixed/updated in opentelemetry 🤞

Mongoose instrumentation does not work for Mongoose >= 7

> Great to hear! Do you want me to create a separate issue for the mongo problem or are you tracking both of those with this one or should I...

Critical Dependency Warning After Upgrading to @sentry/nextjs v8

Hmm, let's see if this: https://github.com/getsentry/sentry-javascript/pull/12081 possibly fixes this 🤔

Redis span ops marked as 'db' instead of 'cache'

So `cachePrefixes` defines a list of prefixes, if any of them is matched we will ensure these spans will show up as cache spans in the UI. So e.g. `{...

URL parameterization not taking into account express routers in v8

Hey, thanks for reporting this. It seems there are still some rough spots in the new express instrumentation. We'll work on updating this to get back to full feature parity...

URL parameterization not taking into account express routers in v8

Ah yeah, and related question: What node version do you use?

feat(node): Ensure express spans have better data

> I feel you rushed 8.x too much. I will wait until a few patches come out. That's totally fair! v7 is still around and is still receiving bug fixes,...

Replacement for Handlers.requestHandler, Handlers.errorHandler, etc.

Thank you for your feedback! I'll try to address the points: 1. The express instrumentation is not strictly needed for errors-only mode, request isolation should also work without it (this...