Logout public/internal#168
Open
constantine2nd wants to merge 5 commits into
Open
Conversation
Previously defaulted to OBPv6.0.0. Updated VITE_OBP_API_DEFAULT_RESOURCE_DOC_VERSION in .env.example and the in-code fallback in src/obp/index.ts. Infrastructure API calls (entitlements, api-collections, consents, resource-docs fetch) are pinned in shared-constants.ts and are unaffected.
Add VITE_OBP_LOGOUT_MODE to control GET /user/logoff behaviour: - public (default): clear the local session, then redirect to the OIDC provider's end_session_endpoint (RP-initiated SSO logout) so the Keycloak/OIDC session is also ended. Falls back to a local redirect when the provider, end_session_endpoint, or id_token is unavailable. - internal: local-only logout, leaving the provider SSO session intact for silent re-login. Unrecognised values warn and default to public. Adds getEndSessionEndpoint() to OAuth2ClientWithConfig and the supporting oauth2 type. Documented in README and .env.example.
- README: new 'OAuth2 / OIDC login providers' section explaining provider discovery via OBP-API /well-known, the both-sides configuration requirement, a step-by-step Login with Google walkthrough, and a migration note for the legacy VITE_OBP_OAUTH2_CLIENT_ID vars (no longer read; rename to VITE_OBP_OIDC_CLIENT_ID). - .env.example: Google provider block now documents how to obtain credentials, the redirect URI to register, and the OBP-API props (oauth2.oidc_provider, oauth2.jwk_set.url) needed on the other side.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.