pinojs / pino-elasticsearch

🌲 load pino logs into Elasticsearch
MIT License
179 stars 67 forks source link

fixed issue where time is a number #35

Closed joeyjiron06 closed 4 years ago

joeyjiron06 commented 5 years ago

Great repo! We've been using this for a couple of weeks and it works great! We are relying on this for a local development. The issue that we are currently facing is that even though our logs have a timestamp, they all get the default timestamp from this library because time is a number in our logs.

Summary of changes

How fast can you get a new release out with this fix?

joeyjiron06 commented 5 years ago

awesome, not sure why the build failed, i see this error. mind taking a look?

resolvconf
11.40s$ travis_apt_get_update
Installing APT Packages
0.48s$ sudo -E apt-get -yq --no-install-suggests --no-install-recommends $(travis_apt_get_options) install oracle-java9-installer
Reading package lists...
Building dependency tree...
Reading state information...
Package oracle-java9-installer is not available, but is referred to by another package.
This may mean that the package is missing, has been obsoleted, or
is only available from another source
E: Package 'oracle-java9-installer' has no installation candidate
apt-get.diagnostics
apt-get install failed
The command "sudo -E apt-get -yq --no-install-suggests --no-install-recommends $(travis_apt_get_options) install oracle-java9-installer" failed and exited with 100 during .
Your build has been stopped.
joeyjiron06 commented 5 years ago

any update on getting this build issue fixed?

joeyjiron06 commented 5 years ago

hey, can i get an update on this? see my coment above about logs

mcollina commented 5 years ago

Would you mind rebasing on top of master? I think I have sorted all the CI issues.

joeyjiron06 commented 5 years ago

Will do

joeyjiron06 commented 4 years ago

updated, let me know when you merge it in. we are waiting for this :)