search

AlgoTrader / betfair

Betfair API-NG library for node.js
96 stars 46 forks source link

Optional Logging #1

Closed mattgi closed 10 years ago

mattgi commented 10 years ago

Are you able to switch off logging by passing something to session?

I tried calling session.stopInvocationLog(); but I receive an error in:

log.debug('Invocation log stopped');

as log is not defined.

I've commented out this line and also any console.logs I've found and it appears to work.. just need to know when to call stopInvocationLog so that it performs as expected.

Thanks

AlgoTrader commented 10 years ago

Thanks for feedback! Can you please fix and send pull request?

mattgi commented 10 years ago

I can take a look in a few weeks. I would like the ability to turn off all logging so it needs more time than I have atm to review the code and remove console.logs and ensure logging can be easily switched on/off as the consumer needs.

AlgoTrader commented 10 years ago

Fixed I beleive