[Freeipa-devel] [PATCH] Spit out info on http connect failure.

Martin Kosek mkosek at redhat.com
Wed Dec 19 08:53:15 UTC 2012


On 12/19/2012 02:28 AM, Simo Sorce wrote:
> Ina code base far far away ... or maybe not.
> ... deep in the bowels of out HTTP connection backend routines ... it so
> happens that we are throwing away an exception.
> 
> So here is an almost one-liner that cheered me up as it unveiled the
> mystery of a client failing to join a replica.
> 
> Cheers,
> Simo.
> 

This is a very useful patch. I know it because I yesterday investigated the
same issue as you and hit the same lack of logging. I just changed the line to:

self.log.info('Connection to %s failed with %s', url, e)

to match our logging style and avoid formatting the logged string unless its
actually used.

ACK. Pushed to master, ipa-3-1.

Martin




More information about the Freeipa-devel mailing list