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

Authentication - a little more options in the UI would help #1465

Open
suratdas opened this issue Dec 28, 2023 · 4 comments
Open

Authentication - a little more options in the UI would help #1465

suratdas opened this issue Dec 28, 2023 · 4 comments

Comments

@suratdas
Copy link
Contributor

suratdas commented Dec 28, 2023

I start FitNesse with passowrds.txt file. I am able to correctly authenticate while accessing secured FitNesse pages. The feature I would be looking forward to are :

  1. Ability to see my username in the UI after I authenticate
  2. A way to logout from the UI (Will be covered separately in a new issue.)
  3. Possibility of changing password for the currently logged in user
  4. A mandatory admin user for user administration
  5. Possibility of admin user to be able to add/delete users via FitNesse UI instead of command line

Please let me know what you think of these features. I can spend some time to see if I can raise a pull request to address some of those.

@fhoeben
Copy link
Collaborator

fhoeben commented Dec 28, 2023

@suratdas I've never used FitNesse's authentication features, but I wouldn't mind merging in one or more PRs to achieve the goals you outline. They seem quite logical when using authentication.

Of course, please do include tests for these features so we can ensure/validate them as part of the normal test sequence.

suratdas added a commit to OrrishLabs/fitnesse that referenced this issue Dec 30, 2023
@suratdas
Copy link
Contributor Author

@fhoeben Thanks. I have raised a pull request. I have to spend quite a bit of time to study and understand the code a little better to implement logout part. Hence, this PR does not include logout. Everything else in this issue is addressed by the PR. Please review and let me know your feedback.

@fhoeben
Copy link
Collaborator

fhoeben commented Feb 19, 2024

Adding users and updating passwords was included in release 20240219.

@suratdas so you want to keep this issue open for 'Logout', or shall we consider this closed?

@suratdas
Copy link
Contributor Author

I have edited the description. I think we can create logout as a separate issue and we can close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants