Dashboard getChartData method fails when Elastic settings not available

[brylie]

While in the process of re-working our settings handling, we get the following error when accessing the dashboard without Elastic settings:

Exception while invoking method 'getChartData' TypeError: Cannot read property 'host' of undefined
     at [object Object].Meteor.methods.getChartData (server/methods/elasticSearch.js:36:44)

[brylie]

@frenchbread we will need to revisit this bug soon. It is affecting @Alapan's work somewhat.

[frenchbread]

There are already couple of checks for settings on elastic-rest package for meteor. Or is this bug on apinf's side?

[brylie]

I think it is in the Apinf getChartData method. Basically, the getChartData method is not properly checking whether the apiUmbrellaConfig object exists prior to using the host attribute. Hence:

Cannot read property 'host' of undefined

[Alapan]

@frenchbread @brylie Is there any update on this issue?

[bajiat]

@frenchbread Could you take this also as a sprint task?

[frenchbread]

@bajiat Self-assigned

[bajiat]

@frenchbread Any update on the progress?

[frenchbread]

Can not reproduce this error anymore. Since we depreciated elastic-rest package in #944. PR contains a logic here to check if ES settings are provided. Should we close this issue then?

[brylie]

Closing, since resolved by PR #944