Missing pastes should not return a 404 but should display a "Paste ID not found" message on the main UI #173
Notifications
Total Time Spent: 1 hour 5 minutes
fen
1 hour 5 minutes
No due date set.
Dependencies
No dependencies set.
Reference: poslop/palette#173
Reference in New Issue
Block a user
Implemented on branch fix/173-paste-not-found (PR #58): missing, expired, and burn-claimed-out paste IDs plus unknown routes now render the main UI with a friendly "Paste ID not found" message (color-coded --err result card, pill radii, no inline scripts/styles, CSP-safe). HTTP status remains 404. go build ./... and go test ./... pass; added TestPasteViewNotFoundFriendly covering the friendly-body path.
Implemented on branch fix/173-paste-not-found (PR #174): missing, expired, and burn-claimed-out paste IDs plus unknown routes now render the main UI with a friendly "Paste ID not found" message (color-coded --err result card, pill radii, no inline scripts/styles, CSP-safe). HTTP status remains 404. go build ./... and go test ./... pass; added TestPasteViewNotFoundFriendly covering the friendly-body path.
QA passed and deployed to palette-dev. Live checks (palette-dev.archfox.org, new build digest sha256:8520c7925472e... confirmed on node via crictl):
QA passed. Verified on palette-dev (dev image digest ffc5ce80f496d, built from merge commit
269acf1): GET /zzzzzz99 returns HTTP 404 with rendered main UI and a friendly 'Paste ID not found' card (headless chromium dump-dom + screenshots at 1400x900 and 375x812, no CSP violations); normal paste create and view unaffected. Closing and deleting fix/173-paste-not-found.QA passed on palette-dev (image sha256:8520c79... = CI dev build of merge commit
269acf1). /nonexistent-paste-xyz and /foo/bar/baz return HTTP 404 with the main UI and the 'Paste ID not found' card (topbar, lockring icon, message, 'Create a new paste' button); no bare '404 page not found' text; rendered DOM verified in headless chromium; no inline styles and no CSP violations in console. Valid paste create/view still works (API create 201, view 200 with content). go build and go test ./... pass locally incl. new TestPasteViewNotFoundFriendly.