getsentry / sentry-javascript

Official Sentry SDKs for JavaScript
https://sentry.io
MIT License
7.97k stars 1.57k forks source link

LogLevel was removed, but is referenced from NestJS-Sentry package #4465

Closed jacoblb64 closed 2 years ago

jacoblb64 commented 2 years ago

Package + Version

Description

Saw this build error when updating to the latest Node SDK version and wanted to flag the probable cause of folks using this export, and encourage the Sentry team and maintainers of the NestJS integration package to work together toward a resolution.

Relevant discussion in this closed issue https://github.com/getsentry/sentry-javascript/issues/4449, and was removed in PR https://github.com/getsentry/sentry-javascript/pull/4317.

LogLevel is used in the basic getting started section of ntegral's popular NestJS wrapper for Sentry: https://github.com/ntegral/nestjs-sentry.

What would the Sentry team recommend using instead? Or is this solely the responsibility of the NesJS integration to solve?

lobsterkatie commented 2 years ago

Hi, @jacoblb64, thanks for writing in!

The Nest SDK is community-supported, so they're the best folks to help with this; I'd suggest opening a similar issue in their repo. That said, if questions come up as you're working with them there, feel free to point them in this direction and we'll be happy to offer advice where we can.

Cheers!

jacoblb64 commented 2 years ago

Sounds good, will chat with folks there, appreciate the help!