Skip to content

Commit

Permalink
feat: Use nest app logger for Terminus per default
Browse files Browse the repository at this point in the history
It will now use the configured NestJS application logger
that has been configured with `app.useLogger(...)`

resolves #2547
  • Loading branch information
BrunnerLivio committed Jul 16, 2024
1 parent 1a655e4 commit 40c6ec5
Show file tree
Hide file tree
Showing 2 changed files with 20 additions and 15 deletions.
4 changes: 0 additions & 4 deletions lib/health-check/logger/default-logger.service.ts

This file was deleted.

31 changes: 20 additions & 11 deletions lib/health-check/logger/logger.provider.ts
Original file line number Diff line number Diff line change
@@ -1,32 +1,41 @@
import { type LoggerService, type Provider, type Type } from '@nestjs/common';
import { DefaultTerminusLogger } from './default-logger.service';
import {
Logger,
type LoggerService,
type Provider,
type Type,
} from '@nestjs/common';

export const TERMINUS_LOGGER = 'TERMINUS_LOGGER';

const NOOP_LOGGER = {
// eslint-disable-next-line @typescript-eslint/no-empty-function
log: () => {},
// eslint-disable-next-line @typescript-eslint/no-empty-function
error: () => {},
// eslint-disable-next-line @typescript-eslint/no-empty-function
warn: () => {},
};

export function getLoggerProvider(
logger?: Type<LoggerService> | boolean,
): Provider<LoggerService> {
// Enable logging
if (logger === true || logger === undefined) {
return {
provide: TERMINUS_LOGGER,
useClass: DefaultTerminusLogger,
useClass: Logger,
};
}

// Disable logging
if (logger === false) {
return {
provide: TERMINUS_LOGGER,
useValue: {
// eslint-disable-next-line @typescript-eslint/no-empty-function
log: () => {},
// eslint-disable-next-line @typescript-eslint/no-empty-function
error: () => {},
// eslint-disable-next-line @typescript-eslint/no-empty-function
warn: () => {},
},
useValue: NOOP_LOGGER,
};
}

// Custom logger
return {
provide: TERMINUS_LOGGER,
useClass: logger,
Expand Down

0 comments on commit 40c6ec5

Please sign in to comment.