Skip to content
This repository has been archived by the owner. It is now read-only.

[dev.icinga.com #2578] With LDAP auth, first login faild #771

Closed
icinga-migration opened this issue May 4, 2012 · 3 comments
Labels
Milestone

Comments

@icinga-migration
Copy link
Member

@icinga-migration icinga-migration commented May 4, 2012

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2578

Created by ThomasClavier on 2012-05-04 13:10:29 +00:00

Assignee: mhein
Status: Resolved (closed on 2012-05-07 14:44:48 +00:00)
Target Version: 1.7
Last Update: 2012-05-07 14:44:48 +00:00 (in Redmine)


Hello

When we use Active Directory (LDAP) authentication, Auth/DispatcherModel whant to save the new user in database, but without setting user_password and user_salt .... that generate an exception. To workaround this issue i have add this 2 lines in Auth/DispatcherModel.class.php :

...
if (is_array($data) {

  • if (!isset($data['user_password'])) $data['user_password'] = '-';
  • if (!isset($data['user_salt'])) $data['user_salt'] = '-';
    $user = new NsmUser();
    ....

II don't know if it's the good solution ... but that work for me :-)

Changesets

2012-05-07 14:42:43 +00:00 by mhein 60423c6

Auth.Dispatcher creates random password

User will always have some password if created through dispatcher fixes #2578
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented May 7, 2012

Updated by mhein on 2012-05-07 12:50:15 +00:00

  • Assigned to set to mhein
  • Target Version set to 1.7
@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented May 7, 2012

Updated by mhein on 2012-05-07 14:44:45 +00:00

Hi.

Could no reproduce the error but added some method which creates some random password to the user if created through Auth.Dispatcher.

Kind regards,
Marius.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

@icinga-migration icinga-migration commented May 7, 2012

Updated by mhein on 2012-05-07 14:44:48 +00:00

  • Status changed from New to Resolved
  • Done % changed from 0 to 100

Applied in changeset 60423c6.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
1 participant
You can’t perform that action at this time.