Skip to content

Commit 24bd3b1

Browse files
committed
refactor: remove user agent tracking from error reporting system
1 parent f3f9d5b commit 24bd3b1

File tree

6 files changed

+4
-10
lines changed

6 files changed

+4
-10
lines changed

reference/vite-reference/src/components/ErrorBoundary.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ export class ErrorBoundary extends Component<Props, State> {
4242
componentName: this.constructor.name,
4343
},
4444
url: window.location.href,
45-
userAgent: navigator.userAgent,
4645
timestamp: new Date().toISOString(),
4746
level: "error",
4847
});

reference/vite-reference/src/components/ErrorFallback.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ export function ErrorFallback({
8484
</summary>
8585
<pre className="mt-3 text-xs overflow-auto max-h-40 text-muted-foreground">
8686
{error.message || error.toString()}
87-
{error.stack && '\n\n' + error.stack}
87+
{error.stack && '\n\n' + error.stack + '\n\n' + error.componentStack}
8888
</pre>
8989
</details>
9090
)}

reference/vite-reference/src/components/RouteErrorBoundary.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@ export function RouteErrorBoundary() {
3030
message: errorMessage,
3131
stack: errorStack,
3232
url: window.location.href,
33-
userAgent: navigator.userAgent,
3433
timestamp: new Date().toISOString(),
3534
source: 'react-router',
3635
error: error,

reference/vite-reference/src/lib/errorReporter.ts

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
interface BaseErrorData {
22
url: string;
3-
userAgent: string;
43
timestamp: string;
54
}
65

@@ -442,7 +441,6 @@ class ErrorReporter {
442441
private createBaseErrorData(): BaseErrorData {
443442
return {
444443
url: window.location.href,
445-
userAgent: navigator.userAgent,
446444
timestamp: new Date().toISOString(),
447445
};
448446
}
@@ -657,7 +655,7 @@ const formatConsoleArgs = (args: unknown[]): string => {
657655

658656
type ImmediatePayload = Pick<
659657
ErrorReport,
660-
"message" | "stack" | "url" | "userAgent" | "timestamp" | "level" | "category"
658+
"message" | "stack" | "url" | "timestamp" | "level" | "category"
661659
>;
662660

663661
const createImmediateErrorPayload = (
@@ -667,7 +665,6 @@ const createImmediateErrorPayload = (
667665
message,
668666
stack: new Error().stack,
669667
url: window.location.href,
670-
userAgent: navigator.userAgent,
671668
timestamp: new Date().toISOString(),
672669
level,
673670
category: categorize(message),

reference/vite-reference/src/main.tsx

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
import '@/lib/errorReporter';
12
import { enableMapSet } from "immer";
23
enableMapSet();
34
import { StrictMode } from 'react'

reference/vite-reference/worker/index.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ import { Env } from './core-utils';
99
export interface ClientErrorReport {
1010
message: string;
1111
url: string;
12-
userAgent: string;
1312
timestamp: string;
1413
stack?: string;
1514
componentStack?: string;
@@ -33,8 +32,7 @@ app.get('/api/health', (c) => c.json({ success: true, data: { status: 'healthy',
3332
app.post('/api/client-errors', async (c) => {
3433
try {
3534
const e = await c.req.json<ClientErrorReport>();
36-
if (!e.message || !e.url || !e.userAgent) return c.json({ success: false, error: 'Missing required fields' }, 400);
37-
console.error('[CLIENT ERROR]', JSON.stringify({ timestamp: e.timestamp || new Date().toISOString(), message: e.message, url: e.url, userAgent: e.userAgent, stack: e.stack, componentStack: e.componentStack, errorBoundary: e.errorBoundary, source: e.source, lineno: e.lineno, colno: e.colno }, null, 2));
35+
console.error('[CLIENT ERROR]', JSON.stringify({ timestamp: e.timestamp || new Date().toISOString(), message: e.message, url: e.url, stack: e.stack, componentStack: e.componentStack, errorBoundary: e.errorBoundary }, null, 2));
3836
return c.json({ success: true });
3937
} catch (error) {
4038
console.error('[CLIENT ERROR HANDLER] Failed:', error);

0 commit comments

Comments
 (0)