On Fri, May 03, 2013 at 04:34:31PM +0200, Jakub Hrozek wrote:
On Fri, May 03, 2013 at 03:08:47PM +0200, Pavel Březina wrote:
> Somehow happened that AD returned empty site name and our new srv
> plugin couldn't cope with that. These patches fix it.
In particular I (or other users testing with that AD) have managed to
break the sites in AD in a very creative way. We have verified that a)
on another AD server the Default-First-Site-Name site is correctly
returned and b) these patches fix the problem and make the AD lookups
gracefully fail over in case the site lookup returns an empty string.
Ack
Pushed to master.