Skip to content

Fix authentication#1

Open
actk wants to merge 1 commit intomasterfrom
feature/fix-authentication-and-oidc-local-server
Open

Fix authentication#1
actk wants to merge 1 commit intomasterfrom
feature/fix-authentication-and-oidc-local-server

Conversation

@actk
Copy link
Copy Markdown

@actk actk commented Feb 24, 2026

Fix authentication: persist basic credentials, fix OIDC token check and local server URL resolution

  • BasicAuthenticator now persists credentials via PasswordSafe and pre-fills username on re-prompt
  • AuthenticationService falls back to BasicAuthenticator when EIS config cannot be loaded
  • OidcAuthenticator: fix inverted access token expiry check (< -> >)
  • OidcAuthenticator: resolve OIDC endpoints through gateway for localhost servers
  • OidcAuthenticator: handle 127.0.0.1 alongside localhost for local server detection
  • CiteckServer: fix protocol detection when host includes a port number
  • ServerRecordsApi: prevent infinite auth retry loop on repeated 401
  • CiteckHttpClient: force HTTP/1.1 to avoid h2 upgrade issues; pass response body in UnauthorizedException
  • UnauthorizedException: accept optional response body for better error context

…nd local server URL resolution

- BasicAuthenticator now persists credentials via PasswordSafe and pre-fills username on re-prompt
- AuthenticationService falls back to BasicAuthenticator when EIS config cannot be loaded
- OidcAuthenticator: fix inverted access token expiry check (< -> >)
- OidcAuthenticator: resolve OIDC endpoints through gateway for localhost servers
- OidcAuthenticator: handle 127.0.0.1 alongside localhost for local server detection
- CiteckServer: fix protocol detection when host includes a port number
- ServerRecordsApi: prevent infinite auth retry loop on repeated 401
- CiteckHttpClient: force HTTP/1.1 to avoid h2 upgrade issues; pass response body in UnauthorizedException
- UnauthorizedException: accept optional response body for better error context
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.

1 participant