search

jhurliman / node-rate-limiter

A generic rate limiter for node.js. Useful for API clients, web crawling, or other tasks that need to be throttled
MIT License
1.5k stars 132 forks source link

npm package name conflict #1

Closed crystalneth closed 12 years ago

crystalneth commented 12 years ago

This is what I was looking for however it can't currently be installed with npm since there is already such a package.

crystalneth commented 12 years ago

Hmm... seems this project was never completed to a working state... lots of invalid references, etc.

jhurliman commented 12 years ago

I renamed the package to "limiter" and published to npm to resolve the conflict. I'll take a look at the invalid references shortly. Thanks for the heads up!

crystalneth commented 12 years ago

I figured this project was probably never completed - there were variables unset and referenced out of scope in the core functions, e.g. maxBurst.

Ended up with my own trivial implementation (didn't need the token system.) Seems like a useful library if you get it working :)

martindale commented 12 years ago

@aliix did you ever make your trivial implementation public? I'm in need of such a library.

jhurliman commented 12 years ago

@martindale @aliix, I fixed up a number of issues with the token bucket and pushed v1.0.0 of "limiter" to npm. Sorry for the inconvenience earlier, there were some bugs introduced when I extracted this library out of a larger project of mine and refactored it.