Merge "Allow us to distinguish errors for logged in users"

This commit is contained in:
jenkins-bot 2019-02-26 22:05:28 +00:00 committed by Gerrit Code Review
commit 4f42e4949d
1 changed files with 4 additions and 2 deletions

View File

@ -8,7 +8,8 @@
* @param {string} [errorUrl] to be logged * @param {string} [errorUrl] to be logged
*/ */
function handleError( errorMessage, lineNumber, columnNumber, errorUrl ) { function handleError( errorMessage, lineNumber, columnNumber, errorUrl ) {
var errorSamplingRate = config.get( 'wgMinervaErrorLogSamplingRate', 0 ), var suffix,
errorSamplingRate = config.get( 'wgMinervaErrorLogSamplingRate', 0 ),
sessionToken = user.sessionId(), sessionToken = user.sessionId(),
EVENT_CLIENT_ERROR_LOG = 'wikimedia.event.WebClientError', EVENT_CLIENT_ERROR_LOG = 'wikimedia.event.WebClientError',
page = M.getCurrentPage(), page = M.getCurrentPage(),
@ -46,7 +47,8 @@
); );
} }
if ( config.get( 'wgMinervaCountErrors' ) ) { if ( config.get( 'wgMinervaCountErrors' ) ) {
mw.track( 'counter.MediaWiki.minerva.WebClientError', 1 ); suffix = user.isAnon() ? '.anon' : '.loggedin';
mw.track( 'counter.MediaWiki.minerva.WebClientError' + suffix, 1 );
} }
} }
// track RL exceptions // track RL exceptions