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

NMS-15780: Admin Password Change Prompt #6375

Conversation

synqotik
Copy link
Contributor

@synqotik synqotik commented Jul 14, 2023

Initial implementation of a password change prompt when the default admin user logs in with username admin, password admin.

  • Update login page to check for “admin/admin” and redirect to Password Gate page

  • Implement Password Gate page which allows user to either change password or skip

  • Clicking “skip” will respect navigating to original target page (if it wasn’t just the login page)

  • Refactored password change servlets to abstract base class since they contain much of the same functionality

  • Fix integration and smoke tests to click “Skip” so they all still work

  • Doesn’t include smoke test for this functionality and UI/UX will need review

Target branch is a feature branch. When smoke test and any UX changes have been implemented and merge to feature branch, will open a separate PR to merge to develop.

External References

Copy link
Contributor

@scthompsonopen scthompsonopen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving because my only comment is a nit. Just a note that this is my first time reviewing jsp based HTML/CSS/JS but it still made sense, and my review is for the FE portions only. Don't know enough Java to review those portions with any competence.


<div class="login-page">
<div class="card login-form rounded">
<div style="padding-bottom: 36px; padding-top: 60px">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super minor nit, but you're mixing inline styles and classes here, and also in the footer. Not a huge deal normally for a small one off but this looks like a new file and there's so much CSS just above this. :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking a look! Will wait til Ben or Christian review and approve due to the Java and server-side stuff, but wanted a FE dev to take a look as well :)

I will update this to use a CSS class here and possibly login.jsp in a subsequent PR.

Copy link
Contributor

@RangerRick RangerRick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just tried it out, everything worked for me.

Skip button makes it easy enough to handle in smoke tests too. 👍

🐿️

@synqotik synqotik merged commit e34822d into features/h33-admin-password-change Jul 14, 2023
3 checks passed
@synqotik synqotik deleted the jira/NMS-15780-admin-pwd-change-prompt branch July 14, 2023 20:03
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