fix: use correct call url in ignore/whitelisturl calls (#1304)

Author: kamilogorek

src/raven.js

@@ -571,6 +571,14 @@ Raven.prototype = {
 
     // stack[0] is `throw new Error(msg)` call itself, we are interested in the frame that was just before that, stack[1]
     var initialCall = isArray(stack.stack) && stack.stack[1];
+
+    // if stack[1] is `Raven.captureException`, it means that someone passed a string to it and we redirected that call
+    // to be handled by `captureMessage`, thus `initialCall` is the 3rd one, not 2nd
+    // initialCall => captureException(string) => captureMessage(string)
+    if (initialCall && initialCall.func === 'Raven.captureException') {
+      initialCall = stack.stack[2];
+    }
+
     var fileurl = (initialCall && initialCall.url) || '';
 
     if (

test/raven.test.js

@@ -3015,6 +3015,30 @@ describe('Raven (public API)', function() {
       assert.isTrue(Raven._send.calledOnce);
     });
 
+    it('should use 3rd frame from stack to get a fileurl if captureMessage was just a redirect from captureException', function() {
+      this.sinon.stub(Raven, '_send');
+      var TraceKitStub = this.sinon.stub(TraceKit, 'computeStackTrace');
+      TraceKitStub.returns({
+        stack: [
+          {url: 'http://example.com', func: 'Raven.captureMessage'},
+          {url: 'http://example.com', func: 'Raven.captureException'},
+          {url: '<anonymous>', func: '?'}
+        ]
+      });
+      Raven._globalOptions.whitelistUrls = {
+        test: function() {
+          return false;
+        }
+      };
+      this.sinon.spy(Raven._globalOptions.whitelistUrls, 'test');
+      this.sinon.spy(Raven, 'captureMessage');
+
+      Raven.captureException('some string');
+
+      assert.isTrue(Raven.captureMessage.calledWith('some string'));
+      assert.isTrue(Raven._globalOptions.whitelistUrls.test.calledWith('<anonymous>'));
+    });
+
     describe('synthetic traces', function() {
       function assertSynthetic(frames) {
         // Raven.captureMessage