On Wed, Jul 19, 2017 at 6:56 PM, lslebodn
<sssd-github-notificat...@fedorahosted.org> wrote:
>   URL: https://github.com/SSSD/sssd/pull/327
> Title: #327: RESPONDERS: Fix terminating idle connections
>
> lslebodn commented:
> """
> On (19/07/17 09:28), fidencio wrote:
>>fidencio commented on this pull request.
>>> @@ -100,11 +100,11 @@ def setup_for_secrets(request):
>>         [secrets]
>>         max_secrets = 10
>>         max_payload_size = 2
>>+        client_idle_timeout = 10
>>
>>@lslebodn: I'm just quoting the commit message as I do believe it answers 
>>your question about the 10 seconds:
>>"The client timeout in the test has to be at least 10 seconds because
>>internally, the responders don't allow a shorter timeout."
>>
> I checked only man page and that's not mention there.
> And it still does not explain why do we need
> to wait 50% longer then `client_idle_timeout`

The reason we have to wait 50% longer than the `client_idle_timeout`
can be find here (please, see the commit message):
https://github.com/SSSD/sssd/commit/560daa14ef013aa14e2aedeea10b07f623d84ec8

And the commit that introduced that the client_idle_timeout has the
minimum value of 10 seconds is this one:
https://github.com/SSSD/sssd/commit/bb79e7559dae451a14150377099e32d6b5159a6c

Hopefully everything is clear now.

>
> LS
>
> """
>
> See the full comment at 
> https://github.com/SSSD/sssd/pull/327#issuecomment-316449831
>
> _______________________________________________
> sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
> To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org
>
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to