
Robert Kehl wrote:
On Tuesday, May 18, 2004 6:21 AM Tyler Hepworth
wrote: I have included a diff file that contains the modifications made to each file. The basic idea is a foreach loop that loops through a list of servers looking for a valid host. The loop quits as soon as the first active host is found. The list is passed in from Config.pm via a reference to a regular array.
Here is what I put in Config.pm
my @hosts =
('server1.example.com','server2.example.com','server3.example.com','etc' );
$Self->{'AuthModule::LDAP::Host'} = \@hosts; $Self->{'Customer::AuthModule::LDAP::Host'} = \@hosts; Host => \@hosts,
Please give me your thoughts on this and whether you think it might be a candidate for inclusion in a future version of OTRS.
Sounds and looks like it should work oob.
One addition, though: If we'd implement it, we'd break existing configurations, which we want to avoid at most costs. Is it possible you implement code to let the user either use a single host or an array?
Ack. Don't want to break things, now do we? I have attached the fix. User's can use the old method $Self->{'AuthModule::...'} = 'ldap@example.com'; or they can pass an array reference: $Self->{'AuthModule::...'} = \@hosts; Easy fix. Hope you like it, Tyler