View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] |
ID | Project | Category | View Status | Date Submitted | Last Update |
0001657 | PacketFence | configuration | public | 2013-06-27 11:22 | 2013-10-08 14:38 |
|
Reporter | Raphux | |
Assigned To | francis | |
Priority | normal | Severity | major | Reproducibility | always |
Status | resolved | Resolution | fixed | |
Platform | | OS | | OS Version | |
Product Version | 4.0.1 | |
Target Version | | Fixed in Version | 4.0.2 | |
|
Summary | 0001657: LDAP test routine doesn't use "port" information |
Description | In the configuration page, users => Sources => Add AD source.
On the page, you can configure the port you want to query. But this parameter is not used when you click the «test» button. It queries 389 (LDAP) by default, event if, for example, you set 3268 (AD Global Catalog Default port), resulting in permanent error. |
Additional Information | I made a small patch, hope that it will be useful. |
Tags | No tags attached. |
|
fixed in git revision | 51f206fe78353b3201ac3380ca1533bec68ddd31 |
fixed in mtn revision | |
|
Attached Files | LDAPSource.pm.patch [^] (517 bytes) 2013-06-27 11:22 [Show Content] [Hide Content]--- lib/pf/Authentication/Source/LDAPSource.pm.ori 2013-06-26 19:17:23.426504436 +0200
+++ lib/pf/Authentication/Source/LDAPSource.pm 2013-06-27 17:04:27.632469585 +0200
@@ -178,7 +178,8 @@
my $logger = Log::Log4perl->get_logger( __PACKAGE__ );
# Connect
- my $connection = Net::LDAP->new($self->{'host'});
+ my $connection = Net::LDAP->new($self->{'host'},
+ port => $self->{port});
if (! defined($connection)) {
$logger->info("Unable to establish LDAP connection.");
|
|