Recached

ThePalaceProject / web-patron
88%
main: 88%

LAST BUILD BRANCH: refs/tags/v0.11.0
DEFAULT BRANCH: main
Repo Added
Token NYLsywhVCNMt2ofO8m0o4lOLYk8jtzujj regen
Build 359 Last
Files 14
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH refs/tags/v0.11.0
branch: SELECT
CHANGE BRANCH
x
Sync Branches

coverage: 88.012%. Remained the same
27374726958

push

github

web-flow 
Support alternate OIDC auth flow via "prompt" param (PP-4429) (#146)

## Description

> [!NOTE]
> The feature described here is basically equivalent to a regular sign
in attempt in the absence of [a supporting change in the Palace Manager
(PR 3452)](https://github.com/ThePalaceProject/circulation/pull/3452).

This adds a "Use a different account" escape hatch to the OIDC error and
cancel screens. Clicking it retries authentication with the
`prompt=select_account` query parameter added to the authorization
request URL, which instructs the OIDC provider to show its account
chooser regardless of any existing session. The normal sign-in flow is
unaffected.

<img width="331" height="329" alt="image"
src="https://github.com/user-attachments/assets/149bbd70-dbb3-46d8-8157-5fb998fcd24d"
/>


Also fixes the OIDC redirect URL, now correctly constructing it using
`URL`/`searchParams` rather than string concatenation.

## Motivation and Context

When a library uses OIDC for sign-in (e.g. Google SSO), the client
auto-redirects to the identity provider. If the user is already signed
into a non-permitted account, authentication fails and they land on the
error screen with only a "Try Again" button — which retries the same
redirect and fails again.

[Jira PP-4429]

## How Has This Been Tested?

- New tests cover the "Use a different account" button appearing in both
the error and cancel states, and verify that clicking it navigates to
the correct URL with `prompt=select_account`.
- Existing OIDC handler tests updated to reflect the corrected URL
format (`?` separator instead of `&`).
- Manual local testing to ensure that the `prompt=select_account` query
param makes it through to the Palace Manager.
- All new and existing tests passed.
- CI tests and checks pass.

## Checklist:

- N/A - I have updated the documentation accordingly.
- [x] All new and existing tests passed.

223 of 258 branches covered (86.43%)

Branch coverage included in aggregate %.

357 of 401 relevant lines covered (89.03%)

31.6 hits per line

Relevant lines Covered
401 RELEVANT LINES 357 COVERED LINES
31.6 HITS PER LINE
Source Files on main

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
27374726958 refs/tags/v0.11.0 Support alternate OIDC auth flow via "prompt" param (PP-4429) (#146) ## Description > [!NOTE] > The feature described here is basically equivalent to a regular sign in attempt in the absence of [a supporting change in the Palace Manager (PR 3452... push web-flow github
88.01
27373069376 main Support alternate OIDC auth flow via "prompt" param (PP-4429) (#146) ## Description > [!NOTE] > The feature described here is basically equivalent to a regular sign in attempt in the absence of [a supporting change in the Palace Manager (PR 3452... push web-flow github
88.01
27372724281 main Update visual prominence of external reader buttons (PP-4355) (#145) ## Description <!--- Describe your changes --> Buttons that open external web readers are now considered as primary actions and are also surfaced on the "My Books" page. The ne... push web-flow github
88.01
27372348596 refactor/prominent-web-reader-button updated Button snapshot Pull #145 PoorBillyPilgrim github
88.01
27311493141 chore/handle-redirect-auth-switching Support alternate OIDC auth flow via "prompt" param (PP-4429) push tdilauro github
88.01
27305596760 refactor/prominent-web-reader-button some cleanup push PoorBillyPilgrim github
88.01
27305195033 refactor/prominent-web-reader-button Moved fulfillable book status text to FulfillmentCard to match new designs from Alissa push PoorBillyPilgrim github
88.01
27304269733 main Show forgot password link (PP-4418) (#144) ## Description <!--- Describe your changes --> Add a "Forgot your password?" link to the CPW sign-in screen, matching the existing behavior in the Palace iOS and Android apps. <img width="362" height="... push web-flow github
88.01
27238065433 feature/reset-pw-link swapped __blank for _blank in ExternalLink component * __blank has no standardized meaning in the HTML spec, so opting for _blank is more appropriate push PoorBillyPilgrim github
88.01
27236601029 feature/reset-pw-link show forgot password link if present for library push PoorBillyPilgrim github
88.01
See All Builds (337)

Badge your Repo: web-patron

We detected this repo isn’t badged! Grab the embed code to the right, add it to your repo to show off your code coverage, and when the badge is live hit the refresh button to remove this message.
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

Refresh