search

ExHammer / hammer-backend-redis

A Redis backend for for the Hammer rate-limiter
https://hexdocs.pm/ExHammer/hammer-backend-redis
MIT License
50 stars 32 forks source link

Bugfix: delete_buckets raises an error #15

Closed ono closed 5 years ago

ono commented 5 years ago

Currently delete_buckets would raise an error as {:ok, [count_deleted, _]} is not the right pattern matching for the pipeline. This PR fixes the issue.

See also: https://redis.io/topics/transactions

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 80.769% when pulling c71f1e1769d2cf65b68fb2e1857875a766236f36 on dicefm:bugfix-delete-buckets into 67c78c73fd4ed297ce3fec264ff53557fe22b8c8 on ExHammer:master.

JuneKelly commented 5 years ago

Very nice, thanks for the PR!

I'll get this merged and released over the weekend.