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

[BUG] nestjs-pino does not support lazy-loaded modules #2069

Open
4 tasks done
casey-chow opened this issue Oct 1, 2024 · 0 comments
Open
4 tasks done

[BUG] nestjs-pino does not support lazy-loaded modules #2069

casey-chow opened this issue Oct 1, 2024 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@casey-chow
Copy link

What is the current behavior?

When I try to lazy-load a module that includes a @InjectPinoLogger decorator, it cannot start.

What is the expected behavior?

Lazy-loading modules should properly inject the logger.

Please provide minimal example repo, not code snippet. Without example repo this issue will be closed.

https://github.com/casey-chow/nestjs-pino-repro

To run and observe failure:

  1. Clone the repository.
  2. npm install
  3. npm start

Please mention other relevant information such as Node.js version and Operating System.

I'm on Node 22.2 and macOS Sonoma.

@casey-chow casey-chow added the bug Something isn't working label Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants