Skip to content

Commit 4e078a0

Browse files
committed
Yank the culprit *before* reversing the frames
1 parent 71e6421 commit 4e078a0

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

src/raven.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -506,11 +506,11 @@ function processException(type, message, fileurl, lineno, frames, options) {
506506
if (globalOptions.ignoreErrors.test(message)) return;
507507

508508
if (frames && frames.length) {
509+
fileurl = frames[0].filename || fileurl;
509510
// Sentry expects frames oldest to newest
510511
// and JS sends them as newest to oldest
511512
frames.reverse();
512513
stacktrace = {frames: frames};
513-
fileurl = frames[0].filename || fileurl;
514514
} else if (fileurl) {
515515
stacktrace = {
516516
frames: [{

test/raven.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -399,10 +399,10 @@ describe('globals', function() {
399399

400400
var frames = [
401401
{
402-
filename: 'http://example.com/file2.js'
402+
filename: 'http://example.com/file1.js'
403403
},
404404
{
405-
filename: 'http://example.com/file1.js'
405+
filename: 'http://example.com/file2.js'
406406
}
407407
], framesFlipped = frames.slice(0);
408408

0 commit comments

Comments
 (0)