Skip to content

Commit 53c0f3f

Browse files
authored
Can set user context via config (fixes #741) (#762)
1 parent d90050d commit 53c0f3f

File tree

2 files changed

+15
-1
lines changed

2 files changed

+15
-1
lines changed

src/raven.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ Raven.prototype = {
102102
if (options) {
103103
each(options, function(key, value){
104104
// tags and extra are special and need to be put into context
105-
if (key === 'tags' || key === 'extra') {
105+
if (key === 'tags' || key === 'extra' || key === 'user') {
106106
self._globalContext[key] = value;
107107
} else {
108108
globalOptions[key] = value;

test/raven.test.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1535,6 +1535,20 @@ describe('Raven (public API)', function() {
15351535
assert.isTrue(Raven._logDebug.called);
15361536
});
15371537

1538+
it('should populate global context for tags, extra, and user', function() {
1539+
Raven.config('//[email protected]/sentry/2', {
1540+
tags: {tag1: 'tagValue1'},
1541+
extra: {extra1: 'extraValue1'},
1542+
user: {name: 'Matt'},
1543+
whitelistUrls: ['not-a-context-value']
1544+
});
1545+
assert.deepEqual(Raven._globalContext, {
1546+
tags: {tag1: 'tagValue1'},
1547+
extra: {extra1: 'extraValue1'},
1548+
user: {name: 'Matt'},
1549+
});
1550+
});
1551+
15381552
describe('whitelistUrls', function() {
15391553
it('should be false if none are passed', function() {
15401554
Raven.config('//[email protected]/2');

0 commit comments

Comments
 (0)