mirror of
https://github.com/snapotter-hq/SnapOtter.git
synced 2026-08-03 07:46:42 +02:00
* feat(analytics): upload web source maps to Sentry + tie release to build Web crash reports were unusable: the bundle ships minified with no source maps uploaded, and every build reported as the frozen APP_VERSION, so a Sentry error showed an unreadable stack under a single release. - Add @sentry/vite-plugin: emit hidden source maps and upload them by debug id when SENTRY_AUTH_TOKEN is present (published Docker build only), then delete the maps so they never ship. No-op for dev and the source archive. - Set the Sentry release from SENTRY_RELEASE / VITE_SENTRY_RELEASE (the Docker build passes the release version), falling back to APP_VERSION. - Relax beforeSend so app bundle frames keep a host-stripped path (Sentry needs it to match the uploaded map) while the instance hostname, error message, and PII stay stripped. Filesystem paths still collapse to the basename. - Wire the Dockerfile (sentry_auth_token build secret + SENTRY_RELEASE arg/env) and the release docker job. * fix(analytics): point source map upload at the snapotter org (project node)
238 lines
8.0 KiB
TypeScript
238 lines
8.0 KiB
TypeScript
// @vitest-environment node
|
|
//
|
|
// Proves the invariant: PostHog and Sentry are NEVER called when
|
|
// analytics is disabled in the baked config.
|
|
import { afterAll, beforeAll, beforeEach, describe, expect, it, vi } from "vitest";
|
|
|
|
const mockPosthogInit = vi.fn(() => ({
|
|
capture: mockCapture,
|
|
startSessionRecording: vi.fn(),
|
|
opt_in_capturing: vi.fn(),
|
|
opt_out_capturing: vi.fn(),
|
|
reset: vi.fn(),
|
|
register: vi.fn(),
|
|
get_distinct_id: vi.fn(() => "test-id"),
|
|
persistence: { disabled: false },
|
|
}));
|
|
const mockCapture = vi.fn();
|
|
|
|
vi.mock("posthog-js", () => ({
|
|
__esModule: true,
|
|
default: { init: mockPosthogInit },
|
|
}));
|
|
|
|
const mockSentryInit = vi.fn();
|
|
const mockBrowserTracingIntegration = vi.fn(() => ({ name: "BrowserTracing" }));
|
|
vi.mock("@sentry/react", () => ({
|
|
init: mockSentryInit,
|
|
browserTracingIntegration: mockBrowserTracingIntegration,
|
|
}));
|
|
|
|
const noop = () => {};
|
|
beforeAll(() => {
|
|
vi.stubGlobal(
|
|
"fetch",
|
|
vi.fn(() => Promise.resolve(new Response("{}", { status: 200 }))),
|
|
);
|
|
process.removeAllListeners("unhandledRejection");
|
|
process.on("unhandledRejection", noop);
|
|
});
|
|
afterAll(() => {
|
|
process.removeListener("unhandledRejection", noop);
|
|
vi.restoreAllMocks();
|
|
});
|
|
|
|
const enabledConfig = {
|
|
enabled: true,
|
|
posthogApiKey: "phc_test",
|
|
posthogHost: "https://ph.test",
|
|
sentryDsn: "https://sentry.test/123",
|
|
sampleRate: 1,
|
|
instanceId: "inst-1",
|
|
};
|
|
|
|
const disabledConfig = {
|
|
enabled: false,
|
|
posthogApiKey: "",
|
|
posthogHost: "",
|
|
sentryDsn: "",
|
|
sampleRate: 0,
|
|
instanceId: "",
|
|
};
|
|
|
|
type AnalyticsModule = typeof import("../../../apps/web/src/lib/analytics");
|
|
let mod: AnalyticsModule;
|
|
|
|
describe("Analytics No-Leak Invariant (baked model)", () => {
|
|
beforeEach(async () => {
|
|
mockPosthogInit.mockClear();
|
|
mockCapture.mockClear();
|
|
mockSentryInit.mockClear();
|
|
vi.resetModules();
|
|
mod = await import("../../../apps/web/src/lib/analytics");
|
|
});
|
|
|
|
describe("when config.enabled is false", () => {
|
|
it("initAnalytics never calls posthog.init", async () => {
|
|
await mod.initAnalytics(disabledConfig);
|
|
expect(mockPosthogInit).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it("initAnalytics never calls Sentry.init", async () => {
|
|
await mod.initAnalytics(disabledConfig);
|
|
expect(mockSentryInit).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it("track() is a silent no-op", async () => {
|
|
await mod.initAnalytics(disabledConfig);
|
|
mod.track("test_event", { key: "value" });
|
|
expect(mockCapture).not.toHaveBeenCalled();
|
|
});
|
|
});
|
|
|
|
describe("when config.enabled is true", () => {
|
|
it("initAnalytics calls posthog.init", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
expect(mockPosthogInit).toHaveBeenCalledOnce();
|
|
});
|
|
|
|
it("track() forwards only allow-listed properties to posthog.capture", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
mod.track("tool_opened", { tool_id: "resize", filename: "secret.png" });
|
|
// filename is free-text and never allow-listed, so it is stripped.
|
|
expect(mockCapture).toHaveBeenCalledWith("tool_opened", { tool_id: "resize" });
|
|
});
|
|
|
|
it("initAnalytics initializes Sentry when sentryDsn provided", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
expect(mockSentryInit).toHaveBeenCalledOnce();
|
|
expect(mockSentryInit).toHaveBeenCalledWith(
|
|
expect.objectContaining({
|
|
dsn: "https://sentry.test/123",
|
|
sendDefaultPii: false,
|
|
}),
|
|
);
|
|
});
|
|
});
|
|
|
|
describe("PII never leaks even when analytics enabled", () => {
|
|
it("Sentry strips the entire user object from events", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
const sentryCall = mockSentryInit.mock.calls.find((call: unknown[]) => call[0]?.beforeSend);
|
|
const beforeSend = sentryCall?.[0].beforeSend;
|
|
if (!beforeSend) return;
|
|
|
|
const event = {
|
|
user: { email: "user@example.com", username: "admin", id: "123" },
|
|
exception: { values: [] },
|
|
};
|
|
const result = beforeSend(event);
|
|
// The hardened model removes the whole user object, not just email/username.
|
|
expect(result.user).toBeUndefined();
|
|
});
|
|
|
|
it("Sentry removes free-text and paths from exception values", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
const sentryCall = mockSentryInit.mock.calls.find((call: unknown[]) => call[0]?.beforeSend);
|
|
const beforeSend = sentryCall?.[0].beforeSend;
|
|
if (!beforeSend) return;
|
|
|
|
const event = {
|
|
exception: {
|
|
values: [
|
|
{
|
|
type: "Error",
|
|
value: "Error processing /tmp/workspace/photo.jpg",
|
|
stacktrace: {
|
|
frames: [{ filename: "/Users/test/project/handler.png" }],
|
|
},
|
|
},
|
|
],
|
|
},
|
|
};
|
|
const result = beforeSend(event);
|
|
// Free-text message is replaced by the bare error type; the path cannot leak.
|
|
expect(result.exception.values[0].value).toBe("Error");
|
|
expect(result.exception.values[0].value).not.toContain("photo.jpg");
|
|
// Filesystem-style paths still collapse to the basename (no dir/username leak).
|
|
expect(result.exception.values[0].stacktrace.frames[0].filename).toBe("handler.png");
|
|
});
|
|
|
|
it("Sentry keeps a host-stripped path for app bundle frames so maps symbolicate", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
const sentryCall = mockSentryInit.mock.calls.find((call: unknown[]) => call[0]?.beforeSend);
|
|
const beforeSend = sentryCall?.[0].beforeSend;
|
|
if (!beforeSend) return;
|
|
|
|
const event = {
|
|
exception: {
|
|
values: [
|
|
{
|
|
type: "TypeError",
|
|
value: "boom",
|
|
stacktrace: {
|
|
frames: [
|
|
{
|
|
filename: "https://files.acme.example/assets/index-abc123.js",
|
|
abs_path: "https://files.acme.example/assets/index-abc123.js",
|
|
lineno: 5,
|
|
colno: 12,
|
|
},
|
|
],
|
|
},
|
|
},
|
|
],
|
|
},
|
|
debug_meta: {
|
|
images: [
|
|
{
|
|
type: "sourcemap",
|
|
code_file: "https://files.acme.example/assets/index-abc123.js",
|
|
debug_id: "11111111-2222-3333-4444-555555555555",
|
|
},
|
|
],
|
|
},
|
|
};
|
|
const result = beforeSend(event);
|
|
const frame = result.exception.values[0].stacktrace.frames[0];
|
|
// Host-less path is kept (Sentry needs it to match the uploaded map); the
|
|
// instance hostname is dropped. Line/column survive for symbolication.
|
|
expect(frame.filename).toBe("/assets/index-abc123.js");
|
|
expect(frame.abs_path).toBe("/assets/index-abc123.js");
|
|
expect(frame.lineno).toBe(5);
|
|
// debug_meta stays consistent with the frame and keeps the debug id.
|
|
expect(result.debug_meta.images[0].code_file).toBe("/assets/index-abc123.js");
|
|
expect(result.debug_meta.images[0].debug_id).toBe("11111111-2222-3333-4444-555555555555");
|
|
// The instance hostname must not appear anywhere in the serialized event.
|
|
expect(JSON.stringify(result)).not.toContain("acme.example");
|
|
});
|
|
|
|
it("Sentry blocks ui.click breadcrumbs", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
const sentryCall = mockSentryInit.mock.calls.find(
|
|
(call: unknown[]) => call[0]?.beforeBreadcrumb,
|
|
);
|
|
const beforeBreadcrumb = sentryCall?.[0].beforeBreadcrumb;
|
|
if (!beforeBreadcrumb) return;
|
|
|
|
expect(beforeBreadcrumb({ category: "ui.click" })).toBeNull();
|
|
});
|
|
|
|
it("Sentry blocks fetch breadcrumbs to file URLs", async () => {
|
|
await mod.initAnalytics(enabledConfig);
|
|
const sentryCall = mockSentryInit.mock.calls.find(
|
|
(call: unknown[]) => call[0]?.beforeBreadcrumb,
|
|
);
|
|
const beforeBreadcrumb = sentryCall?.[0].beforeBreadcrumb;
|
|
if (!beforeBreadcrumb) return;
|
|
|
|
expect(
|
|
beforeBreadcrumb({
|
|
category: "fetch",
|
|
data: { url: "https://example.com/uploads/photo.png" },
|
|
}),
|
|
).toBeNull();
|
|
});
|
|
});
|
|
});
|