Should use different connections for Publish and Subscribe

billcobbler / casbin-redis-watcher

Redis watcher for Casbin https://github.com/casbin/casbin

Apache License 2.0

20 stars 23 forks source link

Closed davidwtf closed 6 years ago

davidwtf commented 6 years ago

I think, it should use different connections for Publish and Subscribe. It will meet this error, when call AddPolicy:

Failure from Redis subscription: ERR only (P)SUBSCRIBE / (P)UNSUBSCRIBE / PING / QUIT allowed in this context

billcobbler commented 6 years ago

You are correct @davidwtf! Thanks for pointing it out. Latest merge should correct that.