Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

logLevel options doesn't seem to do anything #27

Closed
orzarchi opened this issue Jan 26, 2021 · 4 comments
Closed

logLevel options doesn't seem to do anything #27

orzarchi opened this issue Jan 26, 2021 · 4 comments

Comments

@orzarchi
Copy link

orzarchi commented Jan 26, 2021

I set logLevel to LogLevel.Error and used SentryService as my logger.
Expected: Only when logging messages using logger.error will the messages be sent to sentry.
Actual: Log messages of all levels reach sentry.
You can easily see that logLevel isn't being used anywhere in the code if you search this repo

@MickL
Copy link

MickL commented Feb 12, 2021

What did you import? The readme didnt state any import so I was looking around to find the right one for a bit. Seems like there is just: import { LogLevel } from '@sentry/types/dist/loglevel';

@ntegral
Copy link
Owner

ntegral commented May 11, 2021

import { LogLevel } from '@sentry/types';

.........
logLevel: LogLevel.Debug //based on sentry.io loglevel //

@ntegral ntegral closed this as completed May 11, 2021
@abhimanusharma
Copy link

@ntegral getting below error using latest documentation.

node_modules/@ntegral/nestjs-sentry/dist/sentry.interfaces.d.ts:4:10 - error TS2305: Module '"@nestjs/common"' has no exported member 'ConsoleLoggerOptions'.

4 import { ConsoleLoggerOptions } from "@nestjs/common";
           ~~~~~~~~~~~~~~~~~~~~

node_modules/@ntegral/nestjs-sentry/dist/sentry.service.d.ts:1:10 - error TS2305: Module '"@nestjs/common"' has no exported member 'ConsoleLogger'.

1 import { ConsoleLogger } from '@nestjs/common';
           ~~~~~~~~~~~~~

What needs to be change in order to get rid of this error?

@caseychoiniere
Copy link

@abhimanusharma Update NestJS to version 8 or downgrade nestjs-sentry to version 2.x.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants