-
Notifications
You must be signed in to change notification settings - Fork 7
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
Passing an existing instance #27
Comments
Hi, passing an existing instance is not supported atm, you can use the
To integrate it into an Elysia instance, call
Does this work for your scenario? |
If you use custom formatters or serializers in your logger logic, ensure you include those provided by this plugin to maintain its usual behavior: elysia-logger/examples/on-error-with-context.ts Lines 18 to 19 in a1c27e5
They are only set if the Lines 72 to 73 in a1c27e5
|
While that works, its not ideal since it is still creating another Pino instance and any changes we make in one setup need to be copied over to the other. Is there any chance this issue could become a feature request? |
The idea is to replace the old If so, can you please share more information/code snippet on how you create / manage your pino instance? I'll look into adding this feature. |
Hi there,
I already use Pino for my app and was considering adding Elysia. Is there a way to pass my current Pino instance into Elysia?
Thanks.
The text was updated successfully, but these errors were encountered: