Project

General

Profile

Actions
Copy link

Bug #3303

closed

CMS authentication plugin login only loads by username

Added by krileon about 12 years ago. Updated almost 12 years ago.

Status:
Closed
Priority:
High
Assignee:
beat
Target version:
CB 1.8.1
Start date:
24 February 2012
Due date:
% Done:

100%

Estimated time:
4:00 h

Description

When trying to login using LDAP for example the IF check already confirms the user logged in successfully, but we're loading the CB user object by username, which could be stored different from what they supplied for login (often email address). As the user has already confirmed login it should simply load by myId, which will fix all authentication plugin usages as well.

https://www.joomlapolis.com/forum/153-professional-member-support/162511-2516-new-users-always-require-admin-approval?limit=6&start=6

Big thanks to crony for supplying test server and LDAP login.

Files

Download all files
cb.authentication.php (8.64 KB) cb.authentication.php krileon, 24 February 2012 17:24
3303.patch (913 Bytes) 3303.patch krileon, 24 February 2012 17:24
Actions
Copy link Download all files
 #1

Updated by krileon about 12 years ago

Pre-patched CB 1.8 authentication file included.

Actions
Copy link
 #2

Updated by beat about 12 years ago

  • Target version set to CB 1.8.1
Actions
Copy link
 #3

Updated by beat almost 12 years ago

  • Priority changed from Normal to High
Actions
Copy link
 #4

Updated by beat almost 12 years ago

  • Status changed from Resolved to Closed
  • Estimated time set to 4:00 h

committed in r1816. Thanks!

Actions
Copy link

Also available in: Atom PDF