Bug 754951

Summary: Call pwquality library for systemwide password quality checking
Product: [Fedora] Fedora Reporter: Tomas Mraz <tmraz>
Component: system-config-usersAssignee: Nils Philippsen <nphilipp>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: nphilipp, pvrabec
Target Milestone: ---Keywords: FutureFeature, Patch
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Enhancement
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-01-12 17:00:06 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
Proposed patch none

Description Tomas Mraz 2011-11-18 11:01:43 UTC
Created attachment 534374 [details]
Proposed patch

I have written a simple library that should be used systemwide for the password quality checking. The library is named libpwquality and there is also a python wrapper.

I've written a simple patch that calls it if the library is available. I've also added a check against the user name so I needed to modify the function in system-config-users so the user name is passed to it.

Comment 1 Tomas Mraz 2011-11-18 11:06:19 UTC
Note that the package is available on rawhide and the python bindings are in the python-pwquality subpackage, so this requirement should be added to the .spec.

Comment 2 Peter Vrabec 2012-01-11 12:27:37 UTC
Please note, this bug is related to PasswordQualityChecking Feature for Fedora 17. Feature freeze is planned on 2012-02-07. Nils, could you please provide any feedback?

Comment 3 Nils Philippsen 2012-01-12 09:32:43 UTC
Sorry, this fell through the cracks. The patch looks sane, I'll polish it a bit (e.g. clerror ("cracklib error") -> pwerror) before applying. I'll let s-c-users then depend on python-pwquality from F-17 on.

Comment 4 Nils Philippsen 2012-01-12 16:54:05 UTC
Building system-config-users-1.2.112-1.fc17 right now with these changes.