sensu / sensu-dashboard

A dashboard for Sensu, for displaying & managing events & clients.
http://sensuapp.org
MIT License
96 stars 44 forks source link

Stop dashboard process erroring out after 1st page load when api down #152

Closed Fizzadar closed 10 years ago

Fizzadar commented 10 years ago

This stops the dashboards process from dying when the API is down/offline. Page will just reload with error message each time.

portertech commented 10 years ago

"empty_body" will now handle nil and catch empty, works for me.

@amdprophet ?

amdprophet commented 10 years ago

Is this fix working?

portertech commented 10 years ago

@Fizzadar your test isn't using your fix, as it's dieing on line 210, which is now your "if"

Fizzadar commented 10 years ago

@portertech sorry that paste is without the fix, misread your post! @amdprophet Fix is/was working for me all day while fiddling with our Sensu install.

amdprophet commented 10 years ago

@Fizzadar Great! I'll test this out locally and get it merged shortly.