cdmbase / graphql-rabbitmq-subscriptions

A graphql subscriptions implementation using rabbimq and apollo's graphql-subscriptions
MIT License
118 stars 27 forks source link

fix remove all subscribers instead of only the last one #12

Closed xaviertholot closed 5 years ago

xaviertholot commented 6 years ago

Hello,

I have noticed that only the last consumer is cleaned when we unsubscribe... I don't think this was the expected behavior, so I opened a PR.

Please let me know if this is accurate 😄

Thanks

Stradivario commented 5 years ago

Oh my god! I was totally going insane with this bug before 2 years. Thank you very much for this! Hope it will be merged soon!

svengau commented 5 years ago

@veeramarni please, could you publish on npm ? thanks