View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 17693 | Bug reports | Plugins | public | 2021-11-02 21:52 | 2026-10-26 16:40 |
| Reporter | kamkom | Assigned To | c_schmitz | ||
| Priority | none | Severity | partial_block | ||
| Status | resolved | Resolution | fixed | ||
| Product Version | 5.x | ||||
| Summary | 17693: LDAP and 2FA plugin conflict | ||||
| Description | Setting LDAP as default method of authentication disables 2FA input on login screen. | ||||
| Steps To Reproduce | Steps to reproduceEnable LDAP plugin set it as default auth method. Then activate 2FA plugin.
| ||||
| Tags | No tags attached. | ||||
| Bug heat | 28 | ||||
| Complete LimeSurvey version number (& build) | 5.1.17 211025 | ||||
| I will donate to the project if issue is resolved | No | ||||
| Browser | Google Chrome | ||||
| Database type & version | 5.5.5-10.3.29-MariaDB-0+deb10u1 | ||||
| Server OS (if known) | Debian | ||||
| Webserver software & version (if known) | Apache/2.4.38 | ||||
| PHP Version | 7.3.27 | ||||
| Git branch / Pull Request URL | https://github.com/LimeSurvey/LimeSurvey/pull/5403 | ||||
| related to | 17434 | new | Feature requests | TwoFactorAdminLogin work only with AuthDB | |
| has duplicate | 19564 | closed | Bug reports | LDAP and 2FA plugin not running side by side | |
| related to | 20002 | closed | DenisChenu | Bug reports | Create 2FA enforcement setting for 2FA plugin |
|
Not really a conflict in my opinion : https://bugs.limesurvey.org/view.php?id=17434 In general :
The TwoFactorAdminLogin must work like this (in my opinion). |
|
|
I ran into this issue as well - after having LDAP authentication enabled for some time I decided to go ahead and enable the 2FA plugin, only to find myself locked out. I don't disagree with @denischenu but the current functionality is broken. @galads is this something I can help with if someone defines the preferred implementation? I am on CE Version 5.1.10+210913. |
|
|
Surprised this was last updated in 2022. We use Lime Survey (public sector organisation) and the UK are now mandating AD / LDAP integration AND 2FA as a minimum security requirement for logging in to external-facing web apps. Can this be developed, agree with @denischenu |
|
|
I agree it should work like @DenisChenu describes, but right now it doesn't. We have LDAP enabled an AuthDb for the admin user. We'd like to protect the admin user with 2FA, but can't right now, as the 2FA code field is not displayed on the login form. Are there any plans to fix this? |
|
|
My opinion:
Jelle_S : i think you must report separatly here . |
|
|
For our use case it should only apply to authdb, as we only want to protect the admin user with 2FA, so I'm not sure it should be a separate issue? We are experiencing the exact same issue as in the original report. |
|
|
Fix committed to master branch: http://bugs.limesurvey.org/plugin.php?page=Source/view&id=41069 |
|
|
LimeSurvey: master 5e547182 2026-10-22 15:25 Committer: GitHub Details Diff |
Fixed issue 17693: Users with 2FA enabled could not log in when LDAP (or any non-database) authentication was used (05403) |
Affected Issues 17693 |
|
| mod - application/controllers/admin/Authentication.php | Diff File | ||
| mod - application/core/LSUserIdentity.php | Diff File | ||
| mod - application/core/plugins/Authdb/Authdb.php | Diff File | ||
| mod - application/core/plugins/TwoFactorAdminLogin/TwoFactorAdminLogin.php | Diff File | ||
| mod - application/core/plugins/TwoFactorAdminLogin/config.xml | Diff File | ||
| add - application/views/admin/authentication/secondFactor.php | Diff File | ||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2021-11-02 21:52 | kamkom | New Issue | |
| 2021-11-03 08:57 | galads | Assigned To | => galads |
| 2021-11-03 08:57 | galads | Status | new => acknowledged |
| 2021-11-03 09:05 | DenisChenu | Relationship added | related to 17434 |
| 2021-11-03 09:06 | DenisChenu | Note Added: 67069 | |
| 2021-11-03 09:06 | DenisChenu | Bug heat | 0 => 2 |
| 2022-04-07 15:03 | joeiachievedit | Note Added: 68924 | |
| 2022-04-07 15:03 | joeiachievedit | Bug heat | 2 => 4 |
| 2022-11-28 08:55 | DenisChenu | Assigned To | galads => |
| 2022-11-28 08:55 | DenisChenu | Status | acknowledged => confirmed |
| 2022-11-28 08:56 | guest | Bug heat | 4 => 10 |
| 2024-05-14 12:53 | FredJackson | Note Added: 80103 | |
| 2024-05-14 12:53 | FredJackson | Bug heat | 10 => 12 |
| 2024-05-14 12:54 | guest | Bug heat | 12 => 18 |
| 2025-03-12 00:49 | DenisChenu | Relationship added | related to 20002 |
| 2026-02-17 15:43 | Jelle_S | Note Added: 84240 | |
| 2026-02-17 15:43 | Jelle_S | Bug heat | 18 => 20 |
| 2026-02-17 17:11 | DenisChenu | Note Added: 84243 | |
| 2026-02-18 14:11 | Jelle_S | Note Added: 84247 | |
| 2026-10-06 21:49 | c_schmitz | Git branch / Pull Request URL | => https://github.com/LimeSurvey/LimeSurvey/pull/5403 |
| 2026-10-06 21:49 | c_schmitz | Status | confirmed => ready for code review |
| 2026-10-22 15:26 | tibor.pacalat | Assigned To | => tibor.pacalat |
| 2026-10-22 15:26 | tibor.pacalat | Status | ready for code review => resolved |
| 2026-10-22 15:26 | tibor.pacalat | Resolution | open => fixed |
| 2026-10-26 14:39 | c_schmitz | Relationship added | has duplicate 19564 |
| 2026-10-26 14:39 | c_schmitz | Bug heat | 20 => 26 |
| 2026-10-26 16:40 | c_schmitz | Changeset attached | => LimeSurvey master 5e547182 |
| 2026-10-26 16:40 | c_schmitz | Note Added: 86861 | |
| 2026-10-26 16:40 | c_schmitz | Assigned To | tibor.pacalat => c_schmitz |
| 2026-10-26 16:40 | c_schmitz | Bug heat | 26 => 28 |