Redis Adapter does not initialise status

ackintosh / ganesha

:elephant: A Circuit Breaker pattern implementation for PHP applications.

https://ackintosh.github.io/ganesha/

MIT License

585 stars 42 forks source link

Redis Adapter does not initialise status #27

Closed Sertion closed 5 years ago

Sertion commented 6 years ago

The Redis adapter will never be able to loadStatus successfully as it will always exit with Failed to load status. The status is therefor never set. In the same situation in the Memcached adapter it will initialise the status with Ganesha::STATUS_CALMED_DOWN.

The issues can clearly be seen by subscribing to the Ganesha instance.

ackintosh commented 6 years ago

@Sertion Thanks for reporting the issue! I'll investigate the cause of the issue on this weekend.

ackintosh commented 5 years ago

@Sertion I've fixed the issue and released the fix as 0.2.1. 🚀

https://github.com/ackintosh/ganesha/releases/tag/0.2.1

Please give it a try when you have time. Again, thanks for your feedback!!

Sertion commented 5 years ago

Great news! Thank you for looking into the issue!