Skip to content
This repository has been archived by the owner on Jan 15, 2019. It is now read-only.

[dev.icinga.com #2080] Use lowercase before doing username comparisons Part2? #577

Closed
icinga-migration opened this issue Nov 15, 2011 · 4 comments

Comments

@icinga-migration
Copy link

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

Created by silence_le on 2011-11-15 13:00:15 +00:00

Assignee: mhein
Status: Resolved (closed on 2012-01-25 10:48:19 +00:00)
Target Version: 1.6.2
Last Update: 2012-02-14 16:40:35 +00:00 (in Redmine)


Depends on [[#1293]]
I added for the Icinga 1.3.1 the changesets from #1293. But the following occurs:
I have a user with named testTe in my Microsoft Active Directory. Setting up all right, i can do a successfull login with testTe and Password. But in the users settings from icinga-web i can see him imported as testte (lowercase).
If i now login again with testTe i became an Internal Server error. But when i use testte (lowercase) as loginname everthing works as expected. So after Import and successfully first-login i can do only a lowercase login - but some of my Icinga-Web-Users use a case insensitive login name.

I added to
/usr/local/icinga-web/app/modules/AppKit/models/Auth/DispatchModell.class.php
after line 114 (after $this->log....)
$username=strtolower($username);
and i can login with testTe. Don't know if this is the right way.

Changesets

2012-01-25 10:47:41 +00:00 by mhein b6773d9

* Auth.Dispatch lowercase fixes #2080
@icinga-migration
Copy link
Author

Updated by mhein on 2012-01-20 11:42:30 +00:00

  • Category set to Authorization
  • Target Version set to 1.7

@icinga-migration
Copy link
Author

Updated by mhein on 2012-01-25 10:23:42 +00:00

  • Assigned to set to mhein
  • Estimated Hours set to 0.0

@icinga-migration
Copy link
Author

Updated by mhein on 2012-01-25 10:48:19 +00:00

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

Applied in changeset b6773d9.

@icinga-migration
Copy link
Author

Updated by mhein on 2012-02-14 16:40:35 +00:00

  • Target Version changed from 1.7 to 1.6.2

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant