Skip to content

fix(experience): skip step-up interaction check in preview - #9939

Merged
wangsijie merged 1 commit into
masterfrom
wangsijie-skip-step-up-check-in-preview
Oct 30, 2026
Merged

wangsijie merged 1 commit into
masterfrom
wangsijie-skip-step-up-check-in-preview

Conversation

@wangsijie

Copy link
Copy Markdown
Contributor

Summary

The sign-in experience preview in Console sent GET /api/experience/interaction on every load and got a 400 response.

StepUpEntryGuard only trusts the flow cookie's step-up hint when its appId matches the tab's app_id in session storage. The preview iframe is not opened by a login prompt, so it has no app_id. The guard then falls back to reading the interaction, which does not exist in the preview.

The guard now treats the preview as a normal sign-in and renders the entry route without reading the interaction.

Testing

Unit tests

Checklist

  • .changeset
  • unit tests
  • integration tests
  • necessary TSDoc comments

The Console sign-in experience preview has no OIDC interaction, so the
StepUpEntryGuard fallback request to GET /api/experience/interaction always
failed with 400. Treat the preview as a normal sign-in without reading the
interaction.

Amp-Thread-ID: https://ampcode.com/threads/T-01a18b06-5f12-7239-aa41-f11b1d9657d5
@wangsijie
wangsijie enabled auto-merge (squash) October 30, 2026 07:56
@wangsijie
wangsijie merged commit 2e1187b into master Oct 30, 2026
43 of 44 checks passed
@wangsijie
wangsijie deleted the wangsijie-skip-step-up-check-in-preview branch October 30, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants