Bug 25084 - [FR] поддержка pam_userpass
Summary: [FR] поддержка pam_userpass
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: checkpassword-pam (show other bugs)
Version: unstable
Hardware: all Linux
: P3 minor
Assignee: Nobody's working on this, feel free to take it
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-14 14:30 MSK by Afanasov Dmitry
Modified: 2011-02-15 11:14 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Afanasov Dmitry 2011-02-14 14:30:54 MSK
есть предложение в checkpassword-pam добавить поддержку pam_userpass и собирать его с ним.

уже сделано по аналогии с vsftpd, проверено (для себя делал) и готово к отправке: 
http://git.altlinux.org/people/ender/packages/?p=checkpassword-pam.git
Comment 1 Afanasov Dmitry 2011-02-14 14:31:50 MSK
ldv, как участника Owl, прошу прорецензировать - правильно ли вообще реализовано.
Comment 2 Boris Savelev 2011-02-14 16:19:31 MSK
я слабо понимаю зачем это, но acl у Вас есть)
Comment 3 Afanasov Dmitry 2011-02-14 18:42:30 MSK
началось все с того, что у меня checkpassword-pam без tty не хотел работать в режиме echo -e "$user\0$pass\0" | checkpassword-pam -s <service> 3<&0
за некоторое время до этого я как раз изучал, чего такого особенного просит альтовый vsftpd. как оказалось - pam_userpass. чтение readme показало, что pam_userpass создан как раз для случаев, когда авторизировать необходимо без tty.

думая, что это исправит мою проблему, я реализовал pam_userpass для checkpassword-pam. заработало у меня не сразу, как оказалось виноват был не non-interactive login.

но, учитывая, что pam_userpass был сделан как раз для таких случаев, я его решил оставить.

спрашиваю и уведомляю здесь потому, что checkpassword-pam теперь требует required pam_userpass перед required system-auth-first-userpass, а не просто required system-auth, как до этого. что ломает текущее поведение.

если добро, я отправляю в сизиф.
Comment 4 Dmitry V. Levin 2011-02-14 20:33:25 MSK
(In reply to comment #1)
> ldv, как участника Owl, прошу прорецензировать - правильно ли вообще
> реализовано.

Вообще добавление поддержки pam_userpass обычно выглядит просто.
На вид все правильно.  Если собираетесь апстримить, то рекомендую заменить макрос PAM_USERPASS на USE_PAM_USERPASS.
Comment 5 Afanasov Dmitry 2011-02-15 11:14:06 MSK
(В ответ на комментарий №2)
> я слабо понимаю зачем это, но acl у Вас есть)
acl воспользовался :)

заодно добавил ахтунгов в виде README.ALT и сообщения в description, чтоб было что указывать плачущим "а у меня не работает", как с vsftpd было.

(В ответ на комментарий №4)
> Вообще добавление поддержки pam_userpass обычно выглядит просто.
> На вид все правильно.  Если собираетесь апстримить, то рекомендую заменить
> макрос PAM_USERPASS на USE_PAM_USERPASS.
переименовал. в сизиф отправил. закрываю.

а upstream... я пока не думал :)