Squashed commit of logging improvements

[gnomesoftuk]

Please test this and let me know what you think Tony.

In essence what I have done is add more verbose logging and use a production grade logger called winston which allows more flexible logging than the default console logger.

You can add LOGBASEPATH=

to the .env file if you wish to put the logs somewhere else, in my case i've put LOGBASEPATH=/var/log/trades/ in and I have configured a datadog agent to listen to this directory and push it to datadog so I can track my trades without connecting to the server.

Note the path must have a tailing slash.

I just noticed the "reconnect:true" flag made it into this commit - feel free to dump it if you want I am using it without any troubles.

Feel free to change the logging wording if you'd like, I spent a bit of time getting them to look nice but I understand if you and Shannon have other preferences.

I'm happy to share the setup for datadog if you'd like but it sounds like you guys have a pretty decent setup already, I cant really log into servers at work as everything locked down but I can access websites over port 80 so datadog works for me (PS I don't work for these guys lol).

[tony-ho]

@gnomesoftuk Apologies for the (very) delayed response. Thanks for this pull request, it is much appreciated.

I don't think Winston logging is something I want to add at the moment.

Since binance-oco is a command line application, my preference is keep it logging to console and leave it up to the user to redirect output as they wish.

eg. users can use the tee command to redirect output/errors to both the console and file:

binance-oco [arguments...] 2>&1 | tee outputfile.txt

The following changes are good improvements and I would welcome these changes if separated from the Winston logging:

• Timestamps on log messages
• Formatting API responses using JSON.stringify()

The lastPriceUpdate is also a great addition but I feel it could be improved if the 5 minute time window was instead configurable via a command line option, with the default to retain existing behaviour and display every update.

I am going to close this but would be happy to see the above changes opened in a new pull request.

Thanks again for your contribution.