Skip to content
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

Add Cypress tests and documentation #1

Conversation

chutch1122
Copy link

This PR addresses some things requested by the maintainer of NGINX Proxy Manager

  • Cypress automated tests
  • Documentation for setting up SSO with OIDC
  • A more user-friendly error message for when a user does not exist in NPM and they are attempting to sign in via an IdP
  • Fixing a typo in the "hint" description so that the name of "Redirect URL" matches the field label in the modal.

oechsler
oechsler approved these changes Dec 11, 2024
docs/src/setup/index.md Outdated Show resolved Hide resolved
@oechsler oechsler merged commit 2cae60d into oechsler:FEAT/open-id-connect-authentication Dec 11, 2024
@chutch1122
Copy link
Author

@oechsler I just added some additional UI-based end-to-end tests. These tests test logging in with a test user when OIDC is both enabled and disabled

@oechsler
Copy link
Owner

Hey @chutch1122, I completely missed the fact that you made further changes with your last push and merged them directly. If you think a review would still be beneficial, I can revert the changes and reopen this PR. Apologies for the mix-up on my part.

@chutch1122
Copy link
Author

@oechsler Feel free to take a look at it. I'm just opened up a new PR anyway to merge in changes from the upstream develop branch. There were some changes to error messages at some point after the last time develop was pulled in that was causing the build to fail

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