?
avatar joeforjoomla
joeforjoomla
9 Dec 2020

Steps to reproduce the issue

Disable all authentication plugins installed

Expected result

It's not possible to disable all authentication plugins

Actual result

It's possible to disable all of them, as a result it's no longer possible to login to the backend, it's result in an authentication library error because no plugins are enabled

System information (as much as possible)

Joomla 3.9.23

Additional comments

image

avatar joeforjoomla joeforjoomla - open - 9 Dec 2020
avatar joomla-cms-bot joomla-cms-bot - change - 9 Dec 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 9 Dec 2020
avatar brianteeman
brianteeman - comment - 9 Dec 2020

This is a known issue which is why the plugins all have warning text about this. If you can come up with a way to programmatically prevent the last authentication plugin from being disabled that would be awesome. Remember that there can be 3pd authentication plugins as well

avatar bembelimen
bembelimen - comment - 9 Dec 2020

Especially that an easy check "are you the last publish auth plugin" is not enough, because at the end there could be an auth plugin left which still does not allow to login and the site is "broken".

avatar joeforjoomla
joeforjoomla - comment - 9 Dec 2020

Just wanted to point out because a user contacted me for support being blocked to login to the backend. He thought that the problem was one extension of mine... but actually it's the Joomla core that allows a similar situation.

avatar PhilETaylor
PhilETaylor - comment - 16 Dec 2020

I would vote for just changing the warning message to a more helpful "There are no authentication plugins enabled"

It is perfectly reasonable to expect/allow a site to operate with no authentication plugins enabled, should the admin/developer wish. The site might never be logged into and deployed using a deployment scheme from git or ci/cd processes.

The plugin architecture has been abused so much (core/protected plugins that can never be disabled etc) that the original aim (that they could be enabled/disabled) has been lost.

avatar rdeutz rdeutz - close - 20 Dec 2020
avatar rdeutz
rdeutz - comment - 20 Dec 2020

Closing because thie is expected behaviour.

avatar rdeutz rdeutz - change - 20 Dec 2020
Status New Closed
Closed_Date 0000-00-00 00:00:00 2020-12-20 14:09:19
Closed_By rdeutz
avatar PhilETaylor
PhilETaylor - comment - 20 Dec 2020

However this is yet another example of error messages showing the PHP class name and method ... these need to be nuked

Sent from my iPhone

On 20 Dec 2020, at 14:09, Robert Deutz notifications@github.com wrote:


Closing because thie is expected behaviour.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Add a Comment

Login with GitHub to post a comment