search

mappum / webcoin

:globe_with_meridians::moneybag: SPV Bitcoin client for Node.js and the browser
412 stars 60 forks source link

Add the debug package, and debug statements in lib/bridge.js #6

Closed paulkernfeld closed 8 years ago

paulkernfeld commented 8 years ago

This helped me in tracking down issue #5. Do you think this is a good direction? If so, I'm happy to add more debug statements.

mappum commented 8 years ago

Thanks for this. If you change the scope from webcoin to webcoin:bridge I'll merge this. I do think debug statements are good, but there is definitely such a thing as too many. We'll have to mess around with it and find a good balance that gives us meaningful logs without too much noise.

mappum commented 8 years ago

BTW, I plan on breaking the bridge code out into its own repo (named bitcoin-web-bridge).

paulkernfeld commented 8 years ago

Yup, I agree with avoiding too much clutter. Just let me know if this is too noisy.

mappum commented 8 years ago

This is probably OK since the bridge code is mostly run on its own.