Skip to content

feat: add userinfo & end_session endpoints in configure.sh #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

shawnhankim
Copy link
Contributor

Issue Item:

Background:

  • The configure.sh script currently supports to complete the following configuration if an IdP supports OpenID Connect Discovery (usually at the URI /.well-known/openid-configuration).
    • Obtain the URL for jwks_uri or download the JWK file to your NGINX Plus instance
    • Obtain the URL for the authorization endpoint
    • Obtain the URL for the token endpoint
  • NGINX Plus OIDC additionally exposed the endpoints of user information and logout in the PR. Hence they need to be configured by using configure.sh.

Description:

Compatibility:

  • This PR does not block the existing features as it just adds two endpoints.

shawndotkim and others added 5 commits November 15, 2022 13:13
fix: remove changelog to add it later

fix: horizontal alignment
chore: agenda and getting started for README.md

fix: typo

fix: link for README.md

fix: signout and sample API for README.md

fix: comment for post logout and gitignore

fix: space to horizontally aligh variables of key/value

fix: README.md

fix: image of make watch

chore: add a node when Keycloak container isn't working

feat: advanced example of proxy server

chore: fix typo and add a note for two examples for reverse proxy configuration
added doctoc for automatic toc creation
broke getting started info out into new doc
chore: move quick start guide into the repo of nginx-oidc-examples

fix: remove unused files

chore: example link of frontend and backend

fix: back to the original example of configuration

fix: back to the original default vaule of pkce_enable
@shawnhankim
Copy link
Contributor Author

Per discussion with @route443 , I close this PR and have consolidated it into #62. The doc PR will be separately raised. Thanks @route443 😄

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.

3 participants