View Issue Details

This bug affects 1 person(s).
 4
IDProjectCategoryView StatusLast Update
09378Bug reportsAuthenticationpublic2014-12-10 09:21
ReporterDenisChenu Assigned ToDenisChenu  
PrioritynormalSeverityminor 
Status closedResolutionfixed 
Product Version2.05+ 
Fixed in Version2.05+ 
Summary09378: Auth plugin base : lowercase uppercase mix : Not loggued but no error
Description

If you try to connect with an error in lowercase/uppercase : you are not connected, but no error is shown

Steps To Reproduce
  • Create a user like this : user / password (really simple , for testing)
  • Try to connect filling User / password : no error is shown, but not connected
Additional Information

$user = $this->api->getUserByName($username); is Case insensitive (with mysql), somewhere else than AUth plugin, we set to not connected.

Can be fixed by

  • Test if $username==$user->users_name and throw error if not
  • $this->setUsername($user->users_name); before $this->setAuthSuccess($user);
TagsNo tags attached.
Bug heat4
Complete LimeSurvey version number (& build)131130
I will donate to the project if issue is resolvedNo
BrowserFF
Database type & versionMysql 5
Server OS (if known)debian/linux
Webserver software & version (if known)apache
PHP VersionPHP Version 5.4.4

Users monitoring this issue

There are no users monitoring this issue.

Activities

DenisChenu

DenisChenu

2014-11-30 16:24

developer   ~31131

Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=14681

DenisChenu

DenisChenu

2014-11-30 16:30

developer   ~31132

Fix committed to 2.06 branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=14682

DenisChenu

DenisChenu

2014-11-30 16:31

developer   ~31133

Throw error choice : TODO : control if other Auth plugin need update username solution.

c_schmitz

c_schmitz

2014-12-10 09:21

administrator   ~31229

Version 2.05+ Build 141210 released

Related Changesets

LimeSurvey: master 07a2bad7

2014-11-30 15:24:30

DenisChenu

Details Diff
Fixed issue 09378: Auth plugin base : lowercase uppercase mix : Not loggued but no error
Dev: choose the throw error method
Dev: only for mysql ?
Affected Issues
09378
mod - application/core/plugins/Authdb/Authdb.php Diff File

LimeSurvey: 2.06 6d86ea65

2014-11-30 15:30:19

DenisChenu

Details Diff
Fixed issue 09378: Auth plugin base : lowercase uppercase mix : Not loggued but no error
Dev: throw error choice
Dev: For mysql here, but did the other Auth plugin need same ? Then surely 2nd solution.
Affected Issues
09378
mod - application/core/plugins/Authdb/Authdb.php Diff File

Issue History

Date Modified Username Field Change
2014-11-30 16:13 DenisChenu New Issue
2014-11-30 16:24 DenisChenu Changeset attached => LimeSurvey master 07a2bad7
2014-11-30 16:24 DenisChenu Note Added: 31131
2014-11-30 16:24 DenisChenu Assigned To => DenisChenu
2014-11-30 16:24 DenisChenu Resolution open => fixed
2014-11-30 16:30 DenisChenu Changeset attached => LimeSurvey 2.06 6d86ea65
2014-11-30 16:30 DenisChenu Note Added: 31132
2014-11-30 16:31 DenisChenu Note Added: 31133
2014-11-30 16:31 DenisChenu Status new => resolved
2014-11-30 16:31 DenisChenu Fixed in Version => 2.05+
2014-12-10 09:21 c_schmitz Note Added: 31229
2014-12-10 09:21 c_schmitz Status resolved => closed