Don't access document if undefined

Author: benvinegar

src/raven.js

@@ -5,6 +5,8 @@
 // since JSON is required to encode the payload
 var _Raven = window.Raven,
     hasJSON = !!(typeof JSON === 'object' && JSON.stringify),
+    // Raven can run in contexts where there's no document (react-native)
+    hasDocument = typeof document !== 'undefined',
     lastCapturedException,
     lastEventId,
     globalServer,
@@ -390,6 +392,9 @@ Raven.setUser = Raven.setUserContext; // To be deprecated
 function triggerEvent(eventType, options) {
     var event, key;
 
+    if (!hasDocument)
+        return;
+
     options = options || {};
 
     eventType = 'raven' + eventType.substr(0,1).toUpperCase() + eventType.substr(1);
@@ -665,7 +670,7 @@ function now() {
 }
 
 function getHttpData() {
-    if (!document.location || !document.location.href) {
+    if (!hasDocument || !document.location || !document.location.href) {
         return;
     }
 

test/raven.test.js

@@ -1,5 +1,6 @@
 function flushRavenState() {
     hasJSON = !isUndefined(window.JSON);
+    hasDocument = !isUndefined(document);
     lastCapturedException = undefined;
     lastEventId = undefined;
     globalServer = undefined;
@@ -199,23 +200,37 @@ describe('globals', function() {
     });
 
     describe('getHttpData', function() {
-        var data = getHttpData();
+        var data;
 
-        it('should have a url', function() {
-            assert.equal(data.url, window.location.href);
+        before(function () {
+            data = getHttpData();
         });
 
-        it('should have the user-agent header', function() {
-            assert.equal(data.headers['User-Agent'], navigator.userAgent);
+        describe('with document', function() {
+            it('should have a url', function() {
+                assert.equal(data.url, window.location.href);
+            });
+
+            it('should have the user-agent header', function() {
+                assert.equal(data.headers['User-Agent'], navigator.userAgent);
+            });
+
+            it('should have referer header when available', function() {
+                // lol this test is awful
+                if (window.document.referrer) {
+                    assert.equal(data.headers.Referer, window.document.referrer);
+                } else {
+                    assert.isUndefined(data.headers.Referer);
+                }
+            });
         });
 
-        it('should have referer header when available', function() {
-            // lol this test is awful
-            if (window.document.referrer) {
-                assert.equal(data.headers.Referer, window.document.referrer);
-            } else {
-                assert.isUndefined(data.headers.Referer);
-            }
+        describe('without document', function () {
+            it('should return undefined if no document', function () {
+                hasDocument = false;
+                var data = getHttpData();
+                assert.isUndefined(data);
+            });
         });
     });
 

vendor/TraceKit/tracekit.js

@@ -15,7 +15,6 @@ var TraceKit = {
 var _slice = [].slice;
 var UNKNOWN_FUNCTION = '?';
 
-
 /**
  * TraceKit.wrap: Wrap any function in a TraceKit reporter
  * Example: func = TraceKit.wrap(func);
@@ -78,7 +77,8 @@ TraceKit.report = (function reportModuleWrapper() {
     var handlers = [],
         lastArgs = null,
         lastException = null,
-        lastExceptionStack = null;
+        lastExceptionStack = null,
+        hasDocument = document !== 'undefined';
 
     /**
      * Add a crash handler.
@@ -133,6 +133,13 @@ TraceKit.report = (function reportModuleWrapper() {
         }
     }
 
+    function getLocationHref() {
+        if (!hasDocument)
+            return;
+
+        return getLocationHref();
+    }
+
     var _oldOnerrorHandler, _onErrorHandlerInstalled;
 
     /**
@@ -168,7 +175,7 @@ TraceKit.report = (function reportModuleWrapper() {
             location.context = TraceKit.computeStackTrace.gatherContext(location.url, location.line);
             stack = {
                 'message': message,
-                'url': document.location.href,
+                'url': getLocationHref(),
                 'stack': [location]
             };
             notifyHandlers(stack, true);
@@ -512,6 +519,9 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
      * the url, line, and column number of the defined function.
      */
     function findSourceByFunctionBody(func) {
+        if (!hasDocument)
+            return;
+
         var urls = [window.location.href],
             scripts = document.getElementsByTagName('script'),
             body,
@@ -680,7 +690,7 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
         return {
             'name': ex.name,
             'message': ex.message,
-            'url': document.location.href,
+            'url': getLocationHref(),
             'stack': stack
         };
     }
@@ -737,7 +747,7 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
         return {
             'name': ex.name,
             'message': ex.message,
-            'url': document.location.href,
+            'url': getLocationHref(),
             'stack': stack
         };
     }
@@ -847,7 +857,7 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
         return {
             'name': ex.name,
             'message': lines[0],
-            'url': document.location.href,
+            'url': getLocationHref(),
             'stack': stack
         };
     }
@@ -984,7 +994,7 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
         var result = {
             'name': ex.name,
             'message': ex.message,
-            'url': document.location.href,
+            'url': getLocationHref(),
             'stack': stack
         };
         augmentStackTraceWithInitialElement(result, ex.sourceURL || ex.fileName, ex.line || ex.lineNumber, ex.message || ex.description);
@@ -1050,7 +1060,7 @@ TraceKit.computeStackTrace = (function computeStackTraceWrapper() {
         return {
             'name': ex.name,
             'message': ex.message,
-            'url': document.location.href,
+            'url': getLocationHref(),
         };
     }