Fixed angular stacking and not including browser info, bump version

Author: niemyjski

dist/exceptionless.d.ts

@@ -531,5 +531,3 @@ export declare class BrowserStorageProvider implements IStorageProvider {
 export declare class DefaultSubmissionAdapter implements ISubmissionAdapter {
     sendRequest(request: SubmissionRequest, callback?: SubmissionCallback, isAppExiting?: boolean): void;
 }
-export declare function isBrowser(): boolean;
-export declare function browserInit(): void;

dist/exceptionless.js

@@ -2,19 +2,19 @@ angular.module('exceptionless', [])
     .constant('$ExceptionlessClient', exceptionless.ExceptionlessClient.default)
     .factory('exceptionlessHttpInterceptor', ['$q', '$ExceptionlessClient', function ($q, $ExceptionlessClient) {
         return {
-            responseError: function responseError(rejection) {
-                if (rejection.status === 404) {
-                    $ExceptionlessClient.submitNotFound(rejection.config.url);
+            responseError: function responseError(response) {
+                if (response.status === 404) {
+                    $ExceptionlessClient.submitNotFound(response.config.url);
                 }
-                else if (rejection.status !== 401) {
-                    var message = "[" + rejection.status + "] " + (rejection.data && rejection.data.Message ? rejection.data.Message : rejection.config.url);
+                else if (response.status !== 401) {
+                    var message = "[" + response.status + "] " + (response.data && response.data.Message ? response.data.Message : response.config.url);
                     $ExceptionlessClient.createUnhandledException(new Error(message), 'errorHttpInterceptor')
-                        .setManualStackingInfo({ Status: rejection.status, ExceptionType: 'Error', Path: rejection.config.method + ' ' + rejection.config.url })
-                        .setSource(rejection.config.url)
-                        .setProperty('request', rejection.config)
+                        .setSource(response.config.url)
+                        .setCode(response.status)
+                        .setProperty('response', response)
                         .submit();
                 }
-                return $q.reject(rejection);
+                return $q.reject(response);
             }
         };
     }])

dist/exceptionless.js.map

@@ -8,11 +8,11 @@ import { NodeRequestInfoCollector } from './services/NodeRequestInfoCollector';
 import { NodeFileStorageProvider } from './storage/NodeFileStorageProvider';
 import { NodeSubmissionAdapter } from './submission/NodeSubmissionAdapter';
 
-export function isNode(): boolean {
-  return typeof process !== 'undefined';
-}
+(function init() {
+  if (typeof process === 'undefined') {
+    return;
+  }
 
-export function nodeInit() {
   const defaults = Configuration.defaults;
   defaults.environmentInfoCollector = new NodeEnvironmentInfoCollector();
   defaults.errorParser = new NodeErrorParser();
@@ -92,8 +92,4 @@ export function nodeInit() {
   });
 
   (Error as any).stackTraceLimit = Infinity;
-}
-
-if (isNode()) {
-  nodeInit();
-}
+})();

dist/exceptionless.min.js

@@ -11,11 +11,7 @@ import { BrowserStorageProvider } from './storage/BrowserStorageProvider';
 import { DefaultSubmissionAdapter } from './submission/DefaultSubmissionAdapter';
 import { Utils } from './Utils';
 
-export function isBrowser(): boolean {
-  return typeof document !== 'undefined';
-}
-
-export function browserInit() {
+(function init() {
   function getDefaultsSettingsFromScriptTag(): IConfigurationSettings {
     if (!document || !document.getElementsByTagName) {
       return null;
@@ -37,6 +33,10 @@ export function browserInit() {
     builder.submit();
   }
 
+  if (typeof document === 'undefined') {
+    return;
+  }
+
   /*
    TODO: We currently are unable to parse string exceptions.
    function processJQueryAjaxError(event, xhr, settings, error:string): void {
@@ -86,11 +86,7 @@ export function browserInit() {
   // }
 
   (Error as any).stackTraceLimit = Infinity;
-}
-
-if (isBrowser()) {
-  browserInit();
-}
+})();
 
 // tslint:disable-next-line:prefer-const
 declare var $;

dist/exceptionless.min.js.map

@@ -1,13 +1,2 @@
-import { browserInit, isBrowser } from './exceptionless';
-import { isNode, nodeInit } from './exceptionless.node';
-import { ExceptionlessClient } from './ExceptionlessClient';
-
-if (isNode()) {
-  nodeInit();
-  ExceptionlessClient.default.config.log.trace('Using node Exceptionless implementation.');
-} else if (isBrowser()) {
-  browserInit();
-  ExceptionlessClient.default.config.log.trace('Using browser Exceptionless implementation.');
-} else {
-  ExceptionlessClient.default.config.log.error('No Exceptionless implementation was found.');
-}
+import * as ExceptionlessBrowser from './exceptionless';
+import * as ExceptionlessNode from './exceptionless.node';

dist/exceptionless.node.js

@@ -6,18 +6,18 @@ angular.module('exceptionless', [])
   .constant('$ExceptionlessClient', exceptionless.ExceptionlessClient.default)
   .factory('exceptionlessHttpInterceptor', ['$q', '$ExceptionlessClient', ($q, $ExceptionlessClient) => {
     return {
-      responseError: function responseError(rejection) {
-        if (rejection.status === 404) {
-          $ExceptionlessClient.submitNotFound(rejection.config.url);
-        } else if (rejection.status !== 401) {
-          const message = `[${rejection.status}] ${(rejection.data && rejection.data.Message ? rejection.data.Message : rejection.config.url)}`;
+      responseError: function responseError(response) {
+        if (response.status === 404) {
+          $ExceptionlessClient.submitNotFound(response.config.url);
+        } else if (response.status !== 401) {
+          const message = `[${response.status}] ${(response.data && response.data.Message ? response.data.Message : response.config.url)}`;
           $ExceptionlessClient.createUnhandledException(new Error(message), 'errorHttpInterceptor')
-            .setManualStackingInfo({ Status: rejection.status, ExceptionType: 'Error', Path: rejection.config.method + ' ' + rejection.config.url })
-            .setSource(rejection.config.url)
-            .setProperty('request', rejection.config)
+            .setSource(response.config.url)
+            .setCode(response.status)
+            .setProperty('response', response)
             .submit();
         }
-        return $q.reject(rejection);
+        return $q.reject(response);
       }
     };
   }])

dist/exceptionless.node.js.map

@@ -5,7 +5,7 @@ import { IModuleCollector } from './IModuleCollector';
 
 export class DefaultModuleCollector implements IModuleCollector {
   public getModules(context: EventPluginContext): IModule[] {
-    if (document && document.getElementsByTagName) {
+    if (!document || !document.getElementsByTagName) {
       return null;
     }
 
@@ -16,7 +16,7 @@ export class DefaultModuleCollector implements IModuleCollector {
         if (scripts[index].src) {
           modules.push({
             module_id: index,
-            name: scripts[index].src,
+            name: scripts[index].src.split('?')[0],
             version: Utils.parseVersion(scripts[index].src)
           });
         } else if (!!scripts[index].innerHTML) {