Skip to content

test(fe): run the tab and sibling-subdomain session scenarios in a browser - #4289

Open
sea-snake wants to merge 136 commits into
test/session-e2e-staying-signed-infrom
test/session-e2e-tabs-and-siblings
Open

test(fe): run the tab and sibling-subdomain session scenarios in a browser#4289
sea-snake wants to merge 136 commits into
test/session-e2e-staying-signed-infrom
test/session-e2e-tabs-and-siblings

Conversation

@sea-snake

@sea-snake sea-snake commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

The "More than one tab, and sibling subdomains" scenarios of docs/ongoing/session-test-scenarios.md — SHARE-1, SHARE-2, SHARE-3, SHARE-5 and SHARE-6.

One sign-in serves every tab of an origin, and every subdomain of a domain that announces it. A tab or a sibling that was never signed in to arrives working, and loses it when the one that did signs out.

The two sibling scenarios configure the app per subdomain — a derivation origin, and the domain to announce the session across — which authorizeConfig has no field for, so they drive the app themselves. They also declare the siblings as alternative origins first, which the one canister serving all three hosts makes a single write.

@sea-snake
sea-snake marked this pull request as ready for review August 26, 2026 12:56
@sea-snake
sea-snake requested a review from a team as a code owner August 26, 2026 12:56
@sea-snake sea-snake added feature:revocable-app-sessions Design: revocable app sessions feature:silent-reauth-redirect Design: silent re-auth over the redirect transport labels Aug 26, 2026
@sea-snake
sea-snake requested a lite review from Copilot August 26, 2026 12:56
@zeropath-ai

zeropath-ai Bot commented Aug 26, 2026

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to 2002845.

Security Overview
Detected Code Changes
Change Type Relevant files
Enhancement ► src/frontend/tests/e2e-playwright/routes/authorize/app-sessions/tabs-and-siblings.spec.ts
      Add end-to-end tests for multiple tabs and sibling subdomain session sharing

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new Playwright E2E spec to exercise the “More than one tab, and sibling subdomains” session-sharing scenarios from docs/ongoing/session-test-scenarios.md, validating cross-tab and cross-subdomain session continuity and sign-out propagation for the session-based (ICRC-25) flow.

Changes:

  • Introduces a new tabs-and-siblings.spec.ts suite covering multi-tab same-origin session reuse and sign-out visibility.
  • Adds sibling-subdomain scenarios that explicitly configure derivation origin + shared cookie domain and validate silent re-auth from a shared session hint.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@sea-snake
sea-snake force-pushed the test/session-e2e-tabs-and-siblings branch 2 times, most recently from 4006c1a to 33e2c24 Compare August 26, 2026 13:34
@sea-snake
sea-snake force-pushed the test/session-e2e-tabs-and-siblings branch from 33e2c24 to 436e7a8 Compare August 26, 2026 13:36
@sea-snake
sea-snake force-pushed the test/session-e2e-tabs-and-siblings branch from 436e7a8 to b027c00 Compare August 26, 2026 13:53
sea-snake and others added 17 commits September 9, 2026 02:23
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LVi99RYo2jyi2kCurgovNJ
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LVi99RYo2jyi2kCurgovNJ
@sea-snake
sea-snake removed this pull request from stack #4294 September 9, 2026 20:42
@sea-snake
sea-snake added this pull request to stack #4327 September 9, 2026 20:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature:revocable-app-sessions Design: revocable app sessions feature:silent-reauth-redirect Design: silent re-auth over the redirect transport

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants