ExchangeUnion / xud

Exchange Union Daemon 🔁 ⚡️
https://exchangeunion.com
GNU Affero General Public License v3.0
115 stars 44 forks source link

Enhance log to include milliseconds #686

Closed offerm closed 5 years ago

offerm commented 6 years ago

Enhance log output to inlcude milliseconds in each printing.

Today we only should the time up to the seconds.

rsercano commented 5 years ago

This issue seems pretty easy, so I can take over if you're okay with it ? For the miliseconds if we want to print miliseconds then we need to specify format instead of printing current javascript runtime format. I can take this format in config file and provide a default one if not provided.

sangaman commented 5 years ago

Oh yes it had just slipped my mind, go for it if you have time @rsercano. I agree it would be a nice feature.

offerm commented 5 years ago

Suggest we use the same format as LND.

On Wed, Dec 26, 2018 at 3:50 PM Daniel McNally notifications@github.com wrote:

Oh yes it had just slipped my mind, go for it if you have time @rsercano https://github.com/rsercano. I agree it would be a nice feature.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ExchangeUnion/xud/issues/686#issuecomment-449969310, or mute the thread https://github.com/notifications/unsubscribe-auth/AJQ0ct_HdYbaQ61Tkn_xi1AH8Xvr3Hvvks5u836dgaJpZM4YoJhH .

rsercano commented 5 years ago

Now we pick date format from configuration, so everyone can change as how they wish, but I can set the default format as LND if you wish @offerm ?

kilrau commented 5 years ago

default format as LND sounds good! @rsercano