• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

ThePalaceProject / web-patron / 15911132381 / 1
77%
main: 77%

Build:
DEFAULT BRANCH: main
Ran 26 Jun 2025 07:50PM UTC
Files 107
Run time 5s
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

26 Jun 2025 07:49PM UTC coverage: 83.569% (-0.2%) from 83.738%
15911132381.1

push

github

web-flow
PP-2666 : present only the first supported authentication method from auth doc (#68)

## Description

<!--- Describe your changes -->
When navigating to the login screen, a user will no longer be given the
option to select a login method if their auth doc provides more than one
authentication method. Instead, CPW now only grabs the first supported
authentication method provided.

## Motivation and Context

<!--- Why is this change required? What problem does it solve? -->
Authentication is driven and configured by the auth document provided
from the circulation manager, namely by ordering the authentication
methods by priority. The first supported authentication method in the
list should be preferred. Expressly using the first auth method provided
keeps CPW in line with how the CM wants to manage authentication.

Also, removing the user's freedom to choose which authentication method
removes unnecessary ambiguity between some auth methods. For example,
distinguishing between Basic or Basic Token auth may not be a meaningful
choice for a user and does not provide an intuitive UX.

<!--- If it fixes an open issue, please link to the issue here. -->

[Jira PP-2666](https://ebce-lyrasis.atlassian.net/browse/PP-2666)

## How Has This Been Tested?

<!--- Please describe in detail how you tested your changes. -->
<!--- Include details of your testing environment, and the tests you ran
to -->
<!--- see how your change affects other areas of the code, etc. -->

## Checklist:

<!--- Go over all the following points, and put an `x` in all the boxes
that apply. -->
<!--- If you're unsure about any of these, don't hesitate to ask. We're
here to help! -->

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

829 of 1083 branches covered (76.55%)

Branch coverage included in aggregate %.

1414 of 1601 relevant lines covered (88.32%)

29.39 hits per line

Source Files on job 15911132381.1
  • Tree
  • List 107
  • Changed 2
  • Source Changed 0
  • Coverage Changed 2
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses
  • Back to Build 15911132381
  • ccc36c23 on github
  • Prev Job for on main (#15742568006.1)
  • Next Job for on main (#18850433678.1)
  • Delete
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc