Skip to content

[WIP] Fix 401 error handling in session management - #1748

Merged
BigSimmo merged 2 commits into
mainfrom
copilot/issue-overview-fix
Aug 8, 2026
Merged

[WIP] Fix 401 error handling in session management#1748
BigSimmo merged 2 commits into
mainfrom
copilot/issue-overview-fix

Conversation

Copilot AI commented Aug 8, 2026

Copy link
Copy Markdown
Contributor
  • Plan: fix three issues from Copilot review of PR feat(document-viewer): rework the viewer for phone and PWA reading #1741
  • Fix 1: use-signed-image-url.ts — check active before markSessionExpired() to avoid expiring the next user's session on a stale 401
  • Fix 2: pdf-canvas-viewer.tsx — seed manual zoom from current fit scale before first pinch to avoid jump discontinuity
  • Fix 3: canvas-raster-budget.ts — let pixel budget win over density floor so the canvas area never exceeds MAX_CANVAS_PIXELS
  • Tests: stale-401 guard in signed-image.dom.test.tsx; below-floor budget case in canvas-raster-budget.test.ts
  • Typecheck: clean
  • Tests: 5696 passed | 3 skipped | 1 pre-existing failure (pdf-extractor.test.ts, Python not installed in container)

@supabase

supabase Bot commented Aug 8, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.


Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

Copilot AI linked an issue Aug 8, 2026 that may be closed by this pull request
Copilot AI requested a review from BigSimmo August 8, 2026 18:01
@BigSimmo
BigSimmo marked this pull request as ready for review August 8, 2026 18:09
@BigSimmo
BigSimmo enabled auto-merge (squash) August 8, 2026 18:10
@BigSimmo
BigSimmo merged commit 9ab3b73 into main Aug 8, 2026
26 checks passed
@BigSimmo
BigSimmo deleted the copilot/issue-overview-fix branch August 8, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

## Pull request overview

2 participants