Skip to content

Auth Updates for CWMS-CLI Login#1680

Merged
krowvin merged 3 commits intodevelopfrom
cwms-cli-login
Apr 8, 2026
Merged

Auth Updates for CWMS-CLI Login#1680
krowvin merged 3 commits intodevelopfrom
cwms-cli-login

Conversation

@krowvin
Copy link
Copy Markdown
Collaborator

@krowvin krowvin commented Apr 8, 2026

Alters CDA published OpenID Connect metadata and improves local OIDC test coverage

Changes:

  • remove the invalid HTTP auth scheme("openid") from the oIdConnect security scheme so Swagger publishes a correct oIdConnectUrl
  • add a focused test for openID security scheme
  • add debug logging for JWT validation failures to expose the underlying claim/signature issue during local auth setup
  • update local compose seed data so m5hectest has TS ID Creator in SWT, matching the documented/test write-user expectations

Used this in testing

@krowvin krowvin requested a review from MikeNeilson April 8, 2026 04:54
@krowvin krowvin merged commit 20e6a67 into develop Apr 8, 2026
8 of 10 checks passed
@krowvin krowvin deleted the cwms-cli-login branch April 8, 2026 15:17
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.

2 participants