gferrin / bitfinex

node.js wrapper for bitfinex cryptocurrency exchange
26 stars 20 forks source link

use a customizable nonce generator function #23

Closed gmelika closed 9 years ago

gmelika commented 9 years ago

this allows for the module to be used in projects where the same api key is shared among different processes

gferrin commented 9 years ago

Hi gmelika, Thanks for the pull request. I like the idea but I don't like the default nonce generator you replaced mine with. If you set it back to the function that sets it with the date initially and then returns ++nonce I'll be happy to merge your request.

gmelika commented 9 years ago

Fair enough. I've changed the pull request accordingly

gferrin commented 9 years ago

Cool, thanks. I'll add to npm soon.

gmelika commented 9 years ago

thank you.