This revision was automatically updated to reflect the committed changes.
Closed by commit R107:b0ac7bbd6adf: Add readme explaining how kwallet-pam works
(authored by davidedmundson).
REPOSITORY
R107 KWallet PAM Integration
CHANGES SINCE LAST UPDATE
apol accepted this revision.
apol added a comment.
This revision is now accepted and ready to land.
LGTM
REPOSITORY
R107 KWallet PAM Integration
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D20081
To: davidedmundson, #plasma, apol
Cc: apol, plasma-devel, jraleigh, GB_2,
davidedmundson added a comment.
addressed comments
REPOSITORY
R107 KWallet PAM Integration
REVISION DETAIL
https://phabricator.kde.org/D20081
To: davidedmundson, #plasma
Cc: apol, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai,
ali-mohamed, jensreuterberg, abetts,
davidedmundson updated this revision to Diff 55320.
davidedmundson marked 3 inline comments as done.
davidedmundson added a comment.
update
REPOSITORY
R107 KWallet PAM Integration
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D20081?vs=54940=55320
BRANCH
master
REVISION
apol added a comment.
+1 for documenting and the text.
Decorating with some URLs to pam documentation would be nice.
INLINE COMMENTS
> README.txt:3
> +
> +During the pam "auth" (pam_authenticate) stage the module gets the password
> in plaintext
> +It hashes it against a random salt
davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
davidedmundson requested review of this revision.
REVISION SUMMARY
I wasted too long before starting some hacking, I want
to make it easier