@posiwise/common-services 0.2.11 → 0.2.13

package/fesm2022/posiwise-common-services.mjs

@@ -8,7 +8,7 @@ import { DOCUMENT, CommonModule } from '@angular/common';
 import * as i1$2 from '@angular/common/http';
 import { HttpErrorResponse, HttpHeaders } from '@angular/common/http';
 import { throwError, of, BehaviorSubject, tap, timer } from 'rxjs';
-import { catchError, take, switchMap, map, mergeMap, tap as tap$1, distinctUntilChanged } from 'rxjs/operators';
+import { catchError, take, tap as tap$1, switchMap, map, mergeMap, distinctUntilChanged } from 'rxjs/operators';
 import { HelperService } from '@posiwise/helper-service';
 import * as i1$1 from 'ngx-toastr';
 import { getUser, SetUser, UserActionTypes, appReducers } from '@posiwise/app-store';
@@ -1108,9 +1108,12 @@ class AuthService {
                 phonegap: true
             };
         }
-        // Clear all tokens securely
-        this.secureTokenStorage.clearTokens().subscribe();
-        return this.http.delete('session', options);
+        // Send DELETE request with token intact, then clear tokens after response
+        // Use tap to clear tokens without changing the response stream
+        return this.http.delete('session', options).pipe(tap$1(() => {
+            // Clear tokens AFTER logout succeeds
+            this.secureTokenStorage.clearTokens().subscribe();
+        }));
     }
     /**
      * Clear all authentication tokens
@@ -2986,6 +2989,18 @@ class SentryErrorHandler {
                         // We want to ignore those kinds of errors
                         return null;
                     }
+                    // Filter out Edge browser (145.0.0+) synthetic isTrusted events
+                    // These events spam Sentry as "unlabeled events"
+                    const isEdge = /Edg\//.test(navigator.userAgent);
+                    if (isEdge && hint?.originalException) {
+                        const originalException = hint.originalException;
+                        if (originalException &&
+                            typeof originalException === 'object' &&
+                            originalException['isTrusted'] === true) {
+                            // Block Edge synthetic browser events from being sent to Sentry
+                            return null;
+                        }
+                    }
                     // If we started with exception values but filtered them all out, drop the event.
                     // Otherwise Sentry will group it as an "unlabeled event" and spam the issue stream.
                     if (typeof initialExceptionValuesLength === 'number' &&
@@ -3143,6 +3158,13 @@ class SentryErrorHandler {
             scope.setUser({
                 ...user
             });
+            // Filter out Edge browser synthetic events before logging
+            if (this.isEdgeSyntheticEvent(error)) {
+                console.warn('Filtered Edge synthetic event:', error);
+                return;
+            }
+            // Tag the event so it can be properly grouped in Sentry
+            scope.setTag('error_type', this.classifyError(error));
             // Ensure error is properly formatted
             if (error instanceof Error && error.message) {
                 captureException(error);
@@ -3155,6 +3177,48 @@ class SentryErrorHandler {
             }
         });
     }
+    /**
+     * Classifies the error type for proper Sentry tagging
+     * This prevents "unlabeled events" by ensuring all events have a classification
+     */
+    classifyError(error) {
+        if (error instanceof HttpErrorResponse) {
+            return `http_${error.status}`;
+        }
+        if (error instanceof Error) {
+            const name = error.name || 'Error';
+            return name.toLowerCase().replace(/error$/, '') || 'unknown';
+        }
+        if (typeof error === 'string') {
+            return 'string_error';
+        }
+        if (error && typeof error === 'object') {
+            if ('isTrusted' in error) {
+                return 'browser_event';
+            }
+            if ('type' in error) {
+                return `event_${error.type}`.toLowerCase();
+            }
+            return 'object_error';
+        }
+        return 'unknown';
+    }
+    isEdgeSyntheticEvent(error) {
+        // Check if running in Edge browser
+        const isEdge = /Edg\//.test(navigator.userAgent);
+        if (!isEdge) {
+            return false;
+        }
+        // Filter out Edge synthetic DOM events
+        if (error && typeof error === 'object') {
+            const errorObj = error;
+            if ('isTrusted' in errorObj && errorObj['isTrusted'] === true) {
+                // isTrusted browser events in Edge should not be logged
+                return true;
+            }
+        }
+        return false;
+    }
     handleNonStandardError(error) {
         if ('isTrusted' in error) {
             return;