taoensso / carmine

Redis client + message queue for Clojure
https://www.taoensso.com/carmine
Eclipse Public License 1.0
1.15k stars 130 forks source link

message-queue/queue-names is broken (and untested) #299

Closed erichocean closed 4 months ago

erichocean commented 4 months ago

Specifically, the private function qkey requires two parameters, and the implementation of queue-names only provides one.

I also noticed that queue-names is not exercised in any tests AFAICT.

Bug location: https://github.com/taoensso/carmine/blob/master/src/taoensso/carmine/message_queue.clj#L88

ptaoussanis commented 4 months ago

Thanks for the report! This has been fixed on master, and will be addressed in the next release. Apologies for the trouble!