Feature #7587: Javascript validation of passwords on Edit User screen - Foreman

Feature #7587

Javascript validation of passwords on Edit User screen

Added by Dominic Cleal over 5 years ago. Updated almost 2 years ago.

Status:

Closed

Priority:

Normal

Assignee:

Category:

Web Interface

Target version:

1.7.0

Difficulty:

Triaged:

Bugzilla link:

1145626

Pull request:

https://github.com/theforeman/foreman/pull/1832

Fixed in Releases:

Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1145626
++ This bug was initially created as a clone of Bug #1145307 ++

Description of problem:
We don't do any validation or assisting around the creation of passwords on the Edit User page. They can enter passwords that don't match without knowing until they try to change it. There's no strength indicator either.

Version-Release number of selected component (if applicable):
foreman-installer-1.5.0-0.6.RC2.el6ost.noarch
rhel-osp-installer-0.1.10-2.el6ost.noarch
openstack-foreman-installer-2.0.20-1.el6ost.noarch

Additional info:

Associated revisions

Revision 9f2feee3 (diff)
Added by Vanya Jauhal over 5 years ago

Fixes #7587 - Adds client-side password validation

Revision f506c640 (diff)
Added by Dominic Cleal over 5 years ago

refs #7587 - add jquery_pwstrength_bootstrap asset gem

Revision 2a6d13fe (diff)
Added by Dominic Cleal over 5 years ago

refs #7587 - precompile pwstrength JS, move gem from runtime deps

Revision f8de5ce4
Added by Lukas Zapletal over 5 years ago

Merge pull request #380 from domcleal/rpm/develop-7587-pwstrength

refs #7587 - add jquery_pwstrength_bootstrap asset gem

Revision 1222f853 (diff)
Added by Ohad Levy over 5 years ago

Refs #7587 - Refactor password confirmation

confirmation status is now linked to password field
extracted additional missing strings
simplified jquery selectors
switched to onKeyUp built in event
hide progress bar by default
fixed minor css
handle empty pw/confirmation
moved javascript/stylesheet into partial instead of new/edit
moved inline js to onContentLoad event.

Revision db1a98d5 (diff)
Added by Dominic Cleal over 5 years ago

refs #7587 - pwstrength stylesheet is part of app.css, not standalone

History

#1 Updated by Dominic Cleal over 5 years ago

Category set to Web Interface

#2 Updated by b sh over 5 years ago

There are two ways to do that: (1) Write our own Ruby validation method that checks the password strength, or (2) use an open source project, such as plugins.jquery.com/pwstrength-bootstrap/.

Personally, I prefer the latter one.

#3 Updated by Ohad Levy over 5 years ago

b sh wrote:

There are two ways to do that: (1) Write our own Ruby validation method that checks the password strength, or (2) use an open source project, such as plugins.jquery.com/pwstrength-bootstrap/.

Personally, I prefer the latter one.

I tent to agree, but at the moment the url you sent above is not reachable.. :)

#4 Updated by b sh over 5 years ago

Yeah, you're right. Well, anyway - that's its github repo:

https://github.com/ablanco/jquery.pwstrength.bootstrap

;-)

#5 Updated by Ohad Levy over 5 years ago

b sh wrote:

Yeah, you're right. Well, anyway - that's its github repo:

https://github.com/ablanco/jquery.pwstrength.bootstrap

;-)

you probably want to consider something like a rails engine with the assets already, very quick googling showed https://github.com/Skizzk/jquery-pwstrength-bootstrap-rails