Skip to content

[ENG-12119] Filter Sentry client-side errors in Angular-OSF #1059

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
brianjgeiger merged 4 commits into from
Sep 16, 2026
+31 −13
Merged

[ENG-12119] Filter Sentry client-side errors in Angular-OSF #1059

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
07f9d10
feat(sentry): added filtering
nsemets Sep 14, 2026
73a7399
fix(sentry): added beacon error to ignore errors list
nsemets Sep 16, 2026
806034d
Merge remote-tracking branch 'upstream/feature/pbs-26-18' into feat/E…
nsemets Sep 16, 2026
4857cbf
fix(docs): updated docs for sentry filtering
nsemets Sep 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
43 changes: 30 additions & 13 deletions docs/sentry.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,25 +20,42 @@ Three independent checks. An event is dropped if **any** of them match.

Sentry treats each string as a **substring**. `Failed to fetch` also matches `Failed to fetch dynamically imported module`.

| You will stop seeing | Typical cause |
| -------------------------------------------------------------------------------------------------------------------------------------------------------------------- | ------------------------------------------------------------------------------------- |
| Handled unknown error | Sentry could not extract a real Error from Angular |
| Non-Error promise rejection captured… | A promise rejected with `undefined` / `null` / a plain object |
| no elements in sequence | RxJS `EmptyError` (empty observable used with `first()` / `single()`) |
| ResizeObserver loop… | Browser layout warning |
| Failed to fetch | Chrome/Edge: offline, CORS, blocked request, including `api.osf.io` / `addons.osf.io` |
| Load failed | Safari equivalent of failed fetch (including `files.osf.io`) |
| NetworkError when attempting to fetch resource | Firefox equivalent of failed fetch |
| Failed to fetch dynamically imported module / error loading dynamically imported module / Importing a module script failed / ChunkLoadError / Loading chunk … failed | User has an old tab open after a frontend deploy |
| AbortError / The operation was aborted / The user aborted a request | Request cancelled (navigation, timeout, user abort) |
| You will stop seeing | Typical cause |
| ------------------------------------------------------------------- | ------------------------------------------------------------------------------------- |
| Handled unknown error | Sentry could not extract a real Error from Angular |
| Non-Error promise rejection captured… | A promise rejected with `undefined` / `null` / a plain object |
| no elements in sequence | RxJS `EmptyError` (empty observable used with `first()` / `single()`) |
| ResizeObserver loop… | Browser layout warning |
| Failed to fetch | Chrome/Edge: offline, CORS, blocked request, including `api.osf.io` / `addons.osf.io` |
| Load failed | Safari equivalent of failed fetch (including `files.osf.io`) |
| NetworkError when attempting to fetch resource | Firefox equivalent of failed fetch |
| Failed to fetch dynamically imported module | Chrome/Edge: lazy chunk failed to load (often an old tab after deploy) |
| error loading dynamically imported module | Firefox: same lazy-chunk failure |
| Importing a module script failed | Safari: same lazy-chunk failure |
| ChunkLoadError / Loading chunk … failed | Webpack/Vite chunk load failure after deploy |
| AbortError / The operation was aborted / The user aborted a request | Request cancelled (navigation, timeout, user abort) |
| Beacon is not defined | Extension or third-party script; not OSF (`navigator.sendBeacon` is a different API) |

### 2. Script URL (`denyUrls`)

Errors whose stack frames come from a **browser extension**, not from OSF code (`chrome-extension://`, `moz-extension://`, `safari-extension://`, and similar).
Errors whose stack frames come from a **browser extension**, not from OSF code:

- `extensions/`
- `chrome://`
- `chrome-extension://`
- `moz-extension://`
- `safari-extension://`
- `safari-web-extension://`
- `ms-browser-extension://`

### 3. HTTP status below 500 (`beforeSend`)

If the event is an HTTP response (Angular `HttpErrorResponse`, the message `Http failure response for …: 410`, or `Object captured as exception` with HTTP fields) and the status is **0–499**, it is dropped.
If the event is an HTTP response and the status is **0–499**, it is dropped. Status is read from:

- Angular `HttpErrorResponse` (including nested `ngOriginalError` / `rejection` / `cause`)
- `Http failure response for …: 410`
- `Server returned code 404 with body "…"`
- `Object captured as exception with keys: …` or `Non-Error exception captured with keys: …` when the keys look like an HTTP response

| Status | Meaning | Dropped? |
| ------------- | -------------------------------------- | ------------------- |
Expand Down
1 change: 1 addition & 0 deletions src/app/core/helpers/sentry-filter.helper.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ export const SENTRY_IGNORE_ERRORS: (string | RegExp)[] = [
'The user aborted a request',
'ChunkLoadError',
/Loading chunk [\w.-]+ failed/,
'Beacon is not defined',
];

export const SENTRY_DENY_URLS: (string | RegExp)[] = [
Expand Down
Loading