[PATCH] idmap_ad: Retry query_user exactly once if we get TLDAP_SERVER_DOWN

Ralph Böhme slow at samba.org
Tue Jul 11 20:37:20 UTC 2017


On Tue, Jul 11, 2017 at 01:36:21PM -0700, Dustin Howett wrote:
> On Tue, Jul 11, 2017 at 1:02 PM, Ralph Böhme <slow at samba.org> wrote:
> > Does this imply your patch is void? I think it is correct, essentially
> > triggering a reconect of an idle connection that got disconnected by the server
> > (at least that's my understanding of the retry logic).
> >
> 
> Nah. The patch is correct and solves the transient connection loss case
> properly. What led me to submit a patch, though, is the aforementioned *client-
> side* connectivity issue I've been hunting.

ok, in that case: PUSHED! :)

(with the password server change requested by Andrew).

Cheerio!
-slow



More information about the samba-technical mailing list