old data test and delete the old ones by score

[amanda94]

I add a test for my example and change the get function where every time accessing the sorted sets deleting the old data by score

Closes #3

[Kikobeats]

Thanks for that! I'm reviewing it and will be merge soon 🙂

[Kikobeats]

I want to know the opinion @xdmnl because I want to integrate his recent PR: tj/node-ratelimiter#44

In your comment, you are suggesting use zremrangebyscore. Can both improvement life together? do you think that integrate your PR make necessary this one?

[xdmnl]

It is a trade-off between keeping all the data and do 2 reads or keeping only the maximum of timestamps requested by the limiter and then doing only 1 read.
I like the first option (the one that I proposed) better because it reflects reality. If the delay is small enough, the difference won't be noticeable.

In any case, this PR is necessary. It fixes a bug that does not exist in the original library: not expiring out of date timestamps.

[Kikobeats]

Thanks for the feedback guys, released at 1.1.2