Elasticsearch is trying to access loggedin user's "profile" property, which returns null

[jykae]

Found a bug from the recent develop version, I am getting error from elasticSearch.js from line 22.

Exception while invoking method 'getChartData' TypeError: Cannot read property 'profile' of null
I20151215-14:02:54.605(2)? at [object Object].Meteor.methods.getChartData (server/methods/elasticSearch.js:22:1)

[brylie]

@jykae please provide some steps to reproduce. Is this when you deploy? What is the context around this error?

[brylie]

@frenchbread is the chartsLayout template used in any route(s) where anonymous users can access?

We may need to make sure the user is logged in before the getChartData method is envoked, otherwise there needs to be a try catch around the failing code.

[frenchbread]

If i'm recalling correctly, chartsLayout template is not used in any routes. It is used in a dashboard template only. (Did some some search for chartsLayout occurrences within the project folder).

Tested this issue locally by calling Meteor.call("getChartData") and got similar error:

Exception while invoking method 'getChartData' TypeError: Cannot read property 'profile' of null

Seems like when deploying, getChartData gets called for some reason.

Agreed that error handling is needed there. What about checking if user isLoggedIn inside getChartsData method:

if (Meteor.user())
    // continue getting charts data
else
   // terminate

[frenchbread]

Created a hotfix. Please review & test.

This should fix the problem now.

[brylie]

@frenchbread great work!