[SSSD] [PATCH] TESTS: extend PAM responder unit test

Lukas Slebodnik lslebodn at redhat.com
Wed Nov 4 12:19:32 UTC 2015


On (04/11/15 13:14), Pavel Reichl wrote:
>
>
>On 11/04/2015 12:53 PM, Jakub Hrozek wrote:
>>
>>> From f9b5adc160d6c18c429cc4e12276896ccc6f7a6d Mon Sep 17 00:00:00 2001
>>>From: Pavel Reichl <preichl at redhat.com>
>>>Date: Tue, 20 Oct 2015 09:13:53 -0400
>>>Subject: [PATCH 3/3] TESTS: fix typo in PAM service name
>>
>>What is the purpose of this patch? Just to match the service name with
>>what is on Fedora?
>
>My idea is that ssh daemon is the PAM service and not the ssh client no
>matter the system, right? IIRC it does not matter in this test at all but
>it's a typo anyway. I think I briefly asked Sumit about this change and he
>expressed his consensus however I can't find the conversation now so I can't
>recheck.
>
>But If you don't like the change feel free to drop it, it's no doubt a nitpick.
>
You can have any file in /etc/pam.d

+1 for drop

LS


More information about the sssd-devel mailing list