D22122: use pwquality_strerror to get error strings
Harald Sitter
noreply at phabricator.kde.org
Thu Jun 27 14:09:00 BST 2019
sitter created this revision.
sitter added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
sitter requested review of this revision.
REVISION SUMMARY
libpwquality has localized error string mappings, no need to reinvent
the wheel here. the previous mapping was fairly incomplete resulting in
cases where one would get the generic "use another password" option
was shown, which is fairly meh as it can mean literally anything.
should libpwquality for whatever reason fail to provide a string we will
still fall back to the unfortunate generic default. better than nothing
I suppose.
also carry around auxerror. it contains data such as how many digits
would have been reuired. auxerror doesn't need freeing it simply contains
pointers to internals for the benefit of pwquality_strerror
CHANGELOG: Password quality problems are better explained in the UI
TEST PLAN
password 'a' now gets error'd as a palindrom rather than the generic error. other errors such as too-short, no-match continue to be reported properly as well
REPOSITORY
R128 User Manager
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D22122
AFFECTED FILES
src/passworddialog.cpp
src/passworddialog.h
To: sitter, #plasma
Cc: plasma-devel, LeGast00n, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190627/b496a936/attachment-0001.html>
More information about the Plasma-devel
mailing list