Icinga / icingaweb2

A lightweight and extensible web interface to keep an eye on your environment. Analyse problems and act on them.
https://icinga.com/get-started/
GNU General Public License v2.0
809 stars 282 forks source link

CustomUserBackend is loaded too late #5265

Open stevie-sy opened 1 month ago

stevie-sy commented 1 month ago

History

In the meantime, some time has passed as @nilmerg explained me how to create a own custom backend. The requirement was to call Icinga from an application portal (beside the possability via DbBackend) and use its authentication information for the login. Because it's a kind of external backend I should use as role model https://github.com/Icinga/icingaweb2/blob/c4b6e4bdda691820fd3c16841436b2ef796f44b1/library/Icinga/Authentication/User/ExternalBackend.php#L13. Now I have finally managed to do this, but I noticed a few things that I would like to describe here

Describe the bug

During the development I recognized that my module with the own backend is loaded too late and/or the authentifcation call was after putting the credentials into the LoginForm. Depending on the situation icingaweb produces a lot of error messages like: Can't create authentication backend "portal". An exception was thrown: <- Icinga\Exception\ConfigurationError in /usr/share/php/Icinga/Authentication/User/UserBackend.php with message: Authentication configuration for user backend "xxx" defines an invalid backend type. Backend type "xxx" is not supported It doesn't matter in which order I configured my backend and the DbBackend at /etc/icingaweb2/authentication.ini. Because of that I had to debug the complete login sequence.

To Reproduce

The first problem (or actually hurdle) I encourtered is the function https://github.com/Icinga/icingaweb2/blob/c4b6e4bdda691820fd3c16841436b2ef796f44b1/library/Icinga/Authentication/Auth.php#L239 which is triggerd by https://github.com/Icinga/icingaweb2/blob/c4b6e4bdda691820fd3c16841436b2ef796f44b1/library/Icinga/Authentication/Auth.php#L88. The problem what I saw is here, that the function authExternal checks if the backend is a instace of the class ExternalBackend. So If I create my own backend with the implemation of the Interface UserBackendInterface. The function returns false everytime and consequently the authentication fails. Here is the workarround for now that the own backend extends the class ExternalBackend although the most things isn't needed from that class. Maybe to create an own interface, which also will be checked in authExternal instead of the class name? Just to produce nicer code.

The second problem I encourtered is this call https://github.com/Icinga/icingaweb2/blob/c4b6e4bdda691820fd3c16841436b2ef796f44b1/library/Icinga/Authentication/User/UserBackend.php#L91. All installed and activated modules are loaded alphabetically. For example I choose as name for my module "Mybackend". If there are many other modules loaded before like "audit", "icingadb" etc. it takes time, until the own module is loaded. And here happens a big problem: During loading every module and register possible custom backends, following function gets triggerd https://github.com/Icinga/icingaweb2/blob/c4b6e4bdda691820fd3c16841436b2ef796f44b1/library/Icinga/Application/Modules/Module.php#L1374 This function includes the configuration.php and excutes the existing PHP code. If there is a code like (from icingadb-web):

$auth = Auth::getInstance();
$authenticated = Icinga::app()->isWeb() && $auth->isAuthenticated();

the call will fail, because the own modul isn't loaded and the own backend isn't registred yet. This produces the mentioned error message above and happens as long as, until the own module gets loaded! Depending on the number of modules installed, the log file becomes filled with error messages! The only workarround for this is to rename the own module to a little senseless name like "aaamybackend" to provoke the fact that the own module with the own backend is loaded first.

Expected behavior

I know this is a still rare case. Maybe not everybody is crazy like my/us πŸ˜† to create a own backend. So get a solution for this will be low prio I think. For loading in a specific order instead of alphabetical could be to set something like a load order. So that, the call Icinga::app()->getModuleManager()->getLoadedModules() returns the correct and needed order. And for the "problem" with authExternal a solution could be an own interface as I wrote before. But that's only that the code will be nicer, not more. I just wanted to draw attention to it.

Your Environment

Include as many relevant details about the environment you experienced the problem in

nilmerg commented 1 month ago

Hi Stevie ;)

Here is the workarround for now that the own backend extends the class ExternalBackend although the most things isn't needed from that class.

Yep. Would be my suggestion as well, though.

because the own modul isn't loaded and the own backend isn't registred yet.

Have you tried putting the registration of your own backend in run.php? It's run early on, earlier than configuration.php, and the supposed location for hooks (which a custom user backend is somewhat) in general.

stevie-sy commented 1 month ago

Yes I tried run.php as well. Didn't work. But I changed my code so many times until it worked at this time. πŸ˜„ So it could be an other reason at this time. I will test this for you and report back πŸ˜‰

Wintermute2k6 commented 1 month ago

ref/NC/830526

stevie-sy commented 1 month ago

So testet it. And the result is the same like written above: It doesn't matter if the call $this->provideUserBackend is put into run.php nor configuration.php. The own module which register the own backend must be loaded before every other module which will do some checks within Auth::getInstance(); e.g. icingadb-web is doing.

Until there is no load order, you can only choose a name for the module that will ensure that it ranks first in an alphabetical list like the result of this call Icinga::app()->getModuleManager()->getLoadedModules() in https://github.com/Icinga/icingaweb2/blob/c4b6e4bdda691820fd3c16841436b2ef796f44b1/library/Icinga/Authentication/User/UserBackend.php#L91

nilmerg commented 1 month ago

Works, for me. :grin:

I've put this in run.php:

$this->provideUserBackend('test', MyExternalBackend::class);

Which refers to this class:

use Icinga\Authentication\User\ExternalBackend;
use Icinga\Data\ConfigObject;

class MyExternalBackend extends ExternalBackend
{
    public function __construct(ConfigObject $config)
    {

    }

    public static function getRemoteUser($variable = 'REMOTE_USER')
    {
        return 'icingaadmin';
    }
}

And last but not least, an entry in authentication.ini:

[test]
backend = "test"

Now everyone accessing my Web is an admin :tada:

stevie-sy commented 1 month ago

Hmm... intersting ..

Is there a diffrence between:

The last variant you told me month ago ...

Back from vacation I will test it and give feedback

BTW: I know this topic is more then entry level. But would this something for https://github.com/Icinga/icingaweb2-module-training, so it's documented for ohter interesting people?

nilmerg commented 1 month ago

Is there a diffrence between:

No. ::class is just another way of accessing the class path. But this way, it mandates a use … at the top, which lets PHP verify its existence.

And by looking at your class path, MyExternalBackend cannot be the right module name. Please try Myexternalbackend instead.

so it's documented for ohter interesting people?

You already think that this is somewhat quirky to extend ExternalBackend and I agree. So no, I don't want to document this :D

stevie-sy commented 4 weeks ago

No. ::class is just another way of accessing the class path. But this way, it mandates a use … at the top, which lets PHP verify its existence.

Thanks for clarification. Both ways works for me.

And by looking at your class path, MyExternalBackend cannot be the right module name. Please try Myexternalbackend instead.

It was just a copy&paste mistake from your example. My class name in my module is correct - don't worry ☺️

You already think that this is somewhat quirky to extend ExternalBackend and I agree. So no, I don't want to document this

I didn't meant to document to extend ExternalBackend for own backends, I meant the suggestions for $this->provideUserBackend for the docs πŸ˜‰

Now everyone accessing my Web is an admin πŸŽ‰

My module is also working. That is not the problem. But did you check your icingaweb2.log as well in your test? Because as I wrote initially, there are are lot of error messages, until the module with the own backend is loaded. And that is what I mean the own module with the own backend is loaded too late :wink:

nilmerg commented 4 weeks ago

My module is also working. That is not the problem. But did you check your icingaweb2.log as well in your test?

Depending on the situation icingaweb produces a lot of error messages like:

:man_facepalming:

Clearly didn't read this. :roll_eyes:

Yes I have these messages. And I know why. It's because (in my case) businessprocess initiates authentication on its own during init. I knew, and I think I've told you this as well in 2022, that doing this is discouraged. I just didn't knew why until now.

Now we're doing this ourselves :sob:

I meant the suggestions for $this->provideUserBackend for the docs πŸ˜‰

You can always make a suggestion for https://github.com/Icinga/icingaweb2-module-training :D

stevie-sy commented 4 weeks ago

Yes I think you told me this πŸ˜‰

Now we're doing this ourselves

I found this e.g. inrun.php or configuration.php in following modules we are using:

And if I'm correct for every module which is calling Auth::getInstance() there is an error message in icingaweb2.log until the own backend is loaded.

You can always make a suggestion for https://github.com/Icinga/icingaweb2-module-training :D

Yes I know, but there is still the unresolved question with your CLA from our bosses :disappointed_relieved:

nilmerg commented 4 weeks ago

I meant a suggestion in form of an issue, suggesting to add this to the training or introduce one. Not actually introducing this by yourself. ;)