-
Notifications
You must be signed in to change notification settings - Fork 132
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
Winston throws 'Elasticsearch error' with no other information #234
Comments
Any news/information on this? |
I was eventually able to fix this error by just catching all errors in our NestJS application and mutating the error format. It took quite some while to figure out where the exception was actually coming from. |
It still throws this error occasionally. So this is still not fixed unfortunately. |
We eventually just disabled this package and our application never crashes again. Just waiting until it's fixed. |
This case is described in the documentation. I spent a few hours trying to fix this issue and then just realized that the app fails because I don't handle 'error' events for elastic transport and logger instances.
So, to summarise, for the Nest.js application you need:
Hope this information will be helpful. |
Our application sometimes receives a
Elasticsearch error
fromnode_modules/winston-elasticsearch/bulk_writer.js
and specifically on line 134:The error appears to be:
Why and when is such an error usually thrown and what can we do to fix this? This error actually causes our whole container to shut-down. We're using NestJS.
The text was updated successfully, but these errors were encountered: