Skip to content

fix(preview): bound startup readiness - #2489

Merged
chubes4 merged 1 commit into
mainfrom
fix/2489-preview-startup-readiness
Sep 8, 2026
Merged

fix(preview): bound startup readiness#2489
chubes4 merged 1 commit into
mainfrom
fix/2489-preview-startup-readiness

Conversation

@chubes4

@chubes4 chubes4 commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add a bounded 10-second preview startup readiness handshake that follows at most three same-origin redirects.
  • Run the handshake before the external HTTP benchmark so the one-time auto-login redirect is startup work, not a measured request retry.
  • Cover same-origin readiness redirects and reject cross-origin redirect targets before any request reaches them.

Context

Follow-up to #2488.

Verification

  • npm run build:release
  • npm run test:external-http-load
  • npm run test:external-http-load-integration

AI assistance

OpenAI GPT-5.6 Terra via OpenCode assisted with repository inspection, implementation, verification, and PR drafting. Chris Huber authorized the changes and PR.

@chubes4
chubes4 merged commit 4256a29 into main Sep 8, 2026
5 checks passed
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.

1 participant