PrivateCoffee/element-web
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 3

Fix bug that caused crash when analytics HS/IS whitelists not specified

Browse source
This commit is contained in:
Luke Barnard 2018-04-26 15:18:29 +01:00
parent 665e052596
commit 5da62c6ad9
1 changed files with 3 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/Analytics.js
View file

@ -190,6 +190,7 @@ class Analytics {
} }
_setVisitVariable(key, value) { _setVisitVariable(key, value) {
if (this.disabled) return;
this._paq.push(['setCustomVariable', customVariables[key].id, key, value, 'visit']); this._paq.push(['setCustomVariable', customVariables[key].id, key, value, 'visit']);
} }
@ -199,8 +200,8 @@ class Analytics {
const config = SdkConfig.get(); const config = SdkConfig.get();
if (!config.piwik) return; if (!config.piwik) return;
const whitelistedHSUrls = config.piwik.whitelistedHSUrls; const whitelistedHSUrls = config.piwik.whitelistedHSUrls || [];
const whitelistedISUrls = config.piwik.whitelistedISUrls; const whitelistedISUrls = config.piwik.whitelistedISUrls || [];
this._setVisitVariable('User Type', isGuest ? 'Guest' : 'Logged In'); this._setVisitVariable('User Type', isGuest ? 'Guest' : 'Logged In');
this._setVisitVariable('Homeserver URL', whitelistRedact(whitelistedHSUrls, homeserverUrl)); this._setVisitVariable('Homeserver URL', whitelistRedact(whitelistedHSUrls, homeserverUrl));