Upload error card shows the raw MatrixError string including the server URL #213

New Issue

2026-09-18T17:58:17-04:00

jared commented

2026-09-18 17:58:17 -04:00

Seen while verifying #172. When an upload fails permanently (e.g. 413) the upload card prints MatrixError: [413] nope (http://<homeserver>/_matrix/media/v3/upload?filename=upload.png) — the SDK's toString(), URL included.

Map the common cases to one sentence like the call-failure work in #160: 413 → "This file is larger than the server allows (limit N MB)." (the limit is in /_matrix/client/v1/media/config m.upload.size), 429 → "Slow down — try again in a moment.", 5xx/offline after retries → "Couldn't reach the server. Check your connection and retry.", anything else → the server's error text without the URL. Keep the raw string behind Developer Tools / console.

Seen while verifying #172. When an upload fails permanently (e.g. 413) the upload card prints `MatrixError: [413] nope (http://<homeserver>/_matrix/media/v3/upload?filename=upload.png)` — the SDK's `toString()`, URL included. Map the common cases to one sentence like the call-failure work in #160: 413 → "This file is larger than the server allows (limit N MB)." (the limit is in `/_matrix/client/v1/media/config` `m.upload.size`), 429 → "Slow down — try again in a moment.", 5xx/offline after retries → "Couldn't reach the server. Check your connection and retry.", anything else → the server's `error` text without the URL. Keep the raw string behind Developer Tools / console.

jared added the enhancement priority: low ux area: media labels 2026-09-18 17:58:17 -04:00

jared referenced this issue

2026-09-18 17:58:17 -04:00

[QA] §C Upload retry on a flaky network (best-effort) #172

jared commented

2026-09-19 12:40:57 -04:00

Fixed in 6df160a7: describeUploadError() maps the failure to one sentence — 413/M_TOO_LARGE → "This file is larger than the server allows (limit 52.4 MB)." (limit from m.upload.size when known), 429 → "Slow down — try again in a moment.", 401/403 → "The server refused this upload: ", 5xx/transport after the retry loop → "Couldn't reach the server. Check your connection and retry.", any other 4xx → the server's own sentence with the URL stripped. Both the full and compact upload cards use it; the raw MatrixError is still console.warn-ed for debugging. Unit-tested, and verified headless with routed 413 / 403 / 503 responses showing exactly those sentences on the card.

Fixed in `6df160a7`: `describeUploadError()` maps the failure to one sentence — 413/`M_TOO_LARGE` → "This file is larger than the server allows (limit 52.4 MB)." (limit from `m.upload.size` when known), 429 → "Slow down — try again in a moment.", 401/403 → "The server refused this upload: <server text>", 5xx/transport after the retry loop → "Couldn't reach the server. Check your connection and retry.", any other 4xx → the server's own sentence with the URL stripped. Both the full and compact upload cards use it; the raw MatrixError is still `console.warn`-ed for debugging. Unit-tested, and verified headless with routed 413 / 403 / 503 responses showing exactly those sentences on the card.

jared closed this issue

2026-09-19 12:40:58 -04:00

jared referenced this issue from a commit

2026-09-19 12:40:59 -04:00

fix(upload): plain-language upload failure text instead of the raw MatrixError (#213)

jared referenced this issue from a commit

2026-09-19 22:38:43 -04:00

ci(e2e): tier-3 regression suite against a Synapse the job starts itself (#220)

jared referenced this issue

2026-09-19 22:46:15 -04:00

CI: e2e job that starts its own Synapse and runs the UI regression suite (no prod secrets) #220

Sign in to join this conversation.