-
Notifications
You must be signed in to change notification settings - Fork 276
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add LDAP informations to a new user after the registration #282
Conversation
|
||
my $msg = $ldap->bind ( $username, | ||
password => $password, | ||
version => 3); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can use :
my ($connection, $LDAPServer, $LDAPServerPort ) = $self->_connect();
and use $connection instead of $ldap after in your code.
NEWS entry! |
password => $password, | ||
version => 3); | ||
my $searchresult = $ldap->search ( | ||
base => $userdn, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$userdn -> $source->{'basedn'}
You need to rebase your branch before we can merge this. |
e60c7dc
to
7987319
Compare
@@ -24,6 +24,7 @@ New Features | |||
* Add WiFi autoconfiguration for Windows through packetfence-windows-agent | |||
* Added a "Chained" authentication source where a user must first login in order to register by SMS,Email or SponsorEmail | |||
* Added call to the web API from the vlan filters | |||
* Added a way to retrieve user informatione after the first registration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
*zi informatione
my $logger = Log::Log4perl::get_logger('pf::lookup::person'); | ||
my $source = pf::authentication::getAuthenticationSource($source_id); | ||
if (!$source) { | ||
$logger->info("Unable to locate PID '$pid'"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're searching for a source but throwing an error for an unfound PID
Add LDAP informations to a new user after the registration
No description provided.