Strukturfragen Änderungen via src Ordner

[d.tafel]

Hallo,

ich versuche einen eigenen Authenticator einzusetzen.
Dabei mache ich offensichtlich was falsch, hoffe nur eine Kleinigkeit.
So schaut es aus:

- src

- ContaoManager

- Plugin.php

- Security

- MyAuthenticator.php

Plugin.php

<?php
declare(strict_types=1);

namespace App\ContaoManager;

use Contao\CoreBundle\ContaoCoreBundle;
use Contao\ManagerPlugin\Bundle\BundlePluginInterface;
use Contao\ManagerPlugin\Bundle\Config\BundleConfig;
use Contao\ManagerPlugin\Bundle\Parser\ParserInterface;
use Contao\ManagerPlugin\Config\ContainerBuilder;
use Contao\ManagerPlugin\Config\ExtensionPluginInterface;
use App\Security\MyAuthenticator;

class Plugin implements BundlePluginInterface, ExtensionPluginInterface
{
    public function getBundles(ParserInterface $parser): array
    {
        return null;
    }

    public function getExtensionConfig($extensionName, array $extensionConfigs, ContainerBuilder $container): array
    {
        if ('security' !== $extensionName) {
            return $extensionConfigs;
        }

        foreach ($extensionConfigs as &$extensionConfig) {
            if (isset($extensionConfig['firewalls'], $extensionConfig['firewalls']['contao_frontend'])) {
                $extensionConfig['firewalls']['contao_frontend']['anonymous'] = 'lazy';
                $extensionConfig['firewalls']['contao_frontend']['guard']['authenticators'][] = MyAuthenticator::class;
            }
        }

        return $extensionConfigs;
    }
}

MyAuthenticator.php

<?php
declare(strict_types=1);

namespace App\Security;

use Contao\CoreBundle\Framework\ContaoFramework;
use Contao\Date;
use Contao\MemberModel;
use Contao\StringUtil;
use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
use Symfony\Component\HttpFoundation\IpUtils;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Symfony\Component\Security\Core\Exception\AuthenticationException;
use Symfony\Component\Security\Core\Security;
use Symfony\Component\Security\Core\User\UserInterface;
use Symfony\Component\Security\Core\User\UserProviderInterface;
use Symfony\Component\Security\Guard\AbstractGuardAuthenticator;

class MyAuthenticator extends AbstractGuardAuthenticator
{
    /** @var Security */
    private $security;

    /** @var ContaoFramework */
    private $framework;

    /** @var string|null */
    private $requestCondition;

    public function __construct(Security $security, ContaoFramework $framework, ?string $requestCondition)
    {
        $this->security = $security;
        $this->framework = $framework;
        $this->requestCondition = $requestCondition;
    }

    public function supports(Request $request): bool
    {
        // If there already is a user logged in, don't to anything
        if ($this->security->getUser()) {
            return false;
        }
    }

    public function getCredentials(Request $request): string
    {
        return null;
    }

    public function getUser($credentials, UserProviderInterface $userProvider): ?UserInterface
    {
        return null;
    }

    public function checkCredentials($credentials, UserInterface $user): bool
    {
        // LDAP-Elemente zur Authentifizierung
        $ldaprdn  = 'CN='.$credentials->username;
        $ldappass = $credentials->password;
System::log('Test 8', __METHOD__, TL_ACCESS);
        // Verbinden mit dem LDAP-Server
        $ldapconn = ldap_connect("ldap://unternehmerverbaende.de")
            or die("Keine Verbindung zum LDAP-Server möglich.");

        if ($ldapconn) {
            // Anmelden am LDAP-Server (als Member)
            $ldapbind = ldap_bind($ldapconn, $ldaprdn, $ldappass);
            // Überprüfung der Authentifizierung
            if ($ldapbind) {
                return true;
            } else {
                return false;
            }
        }
    }

    public function onAuthenticationSuccess(Request $request, TokenInterface $token, $providerKey): ?Response
    {
        return null;
    }

    public function onAuthenticationFailure(Request $request, AuthenticationException $exception): ?Response
    {
        return null;
    }

    public function supportsRememberMe(): bool
    {
        return false;
    }

    public function start(Request $request, AuthenticationException $authException = null): ?Response
    {
        return null;
    }
}

wird nicht geladen, kein Logeintrag bei Loginversuch, dieser sollte doch feuern bei jeden versuch ab Erfolg oder nicht?

root composer.json

...
    "autoload": {
        "psr-4": {
            "App\\": "src/",
            "AppBundle\\": "src/AppBundle/"
        }
    }

AppBundle ist für eine andere Erweiterung, welche unbekannte User schon bestens abfängt nach meinen Anpassungen.
Vielen Dank

[Spooky]

Der Ordner lautet src/, nicht app/ - dir fehlen außerdem noch die richtigen Namespace Definitionen (siehe dir die Beispiele in der Entwicklerdoku an).

[d.tafel]

Danke für die Rückmeldung.

habe den Beitrag angepasst. Noch klappt etwas nicht.
Die Plugin.php wird geladen, konnte einen Fehler erzeugen.

MyAuthenticator.php bin ich nicht sicher. Ist der Namespace so ausreichend? Oder muss es in einen Unterordner?

[Spooky]

Sofern du PSR-4 verstehst ist alles dir überlassen . Aber üblich wäre folgender Ort für deinen Authenticator:

src/Security/MyAuthenticator.php

Und somit wäre das dann folgender Namespace für diese Klasse:

App\Security

Und somit wäre das folgender FQCN für deinen Authenticator:

App\Security\MyAuthenticator

[d.tafel]

Habe dein Feedback umgesetzt und oben wieder hinterlegt. Es kommt leider immer noch kein Log an auch keine Fehlermeldung.

[d.tafel]

Warum gelingt der Login obwohl hier false zurück gegeben wird:

    public function checkCredentials($credentials, UserInterface $user): bool
    { 
System::log('Test', __METHOD__, TL_ACCESS);
return false;
} 

Der Log wird angezeigt.
Wenn nicht hier, wo kann ich den Login als nicht erfolgreich deklarieren. (meldung fehlgeschlagen (Benutzernamen und Passwörter unterscheiden zwischen Groß- und Kleinschreibung))

[d.tafel]

ok, das true false, wandert weiter runter

    public function onAuthenticationFailure(Request $request, AuthenticationException $exception): ?Response
    {
        System::log('Test Fail', __METHOD__, TL_ACCESS);
        return null;
    } 

wie schaut der Response aus, damit die typische fehlermeldung kommt beim Login Formular?

Dieses allgemeine Beispiel geht natürlich nicht, aber vielleicht mir 1,2 Änderungen...

$request = System::getContainer()->get('request_stack')->getCurrentRequest();
        $request->getSession()->set(Security::AUTHENTICATION_ERROR, $exception);
        return new RedirectResponse(
            $this->router->generate('app_login')
        ); 

[Spooky]

Da du das Login-Formular ansprichst: vielleicht ist ein Custom Authenticator gar nicht das richtige für dich? Erkläre mal was genau du erreichen möchtest.

[d.tafel]

Das Passwort soll nicht von Contao geprüft werden, sondern live am Ldap Server.

if($request->request->get('FORM_SUBMIT')  == 'tl_login_125'){ 

damit ging ich jetzt sicher das mein code nur beim Formular verwendet wird.

mit dem HOOK importUser, konnte ich bisher unbekannte Member nach einer Ldap Authentifizierung anlegen lassen und einloggen.
Ich fand bisher aber keine gute Lösung, folge Logins auch vom Ldap Authentifizieren zu lassen.

Symfony bietet viel, die Beispile auf Contao zu übertragen ist noch eine Herausfoderung.
Bin gespannt auf deine Ideen

[d.tafel]

Ob es genau das Richtige ist, kein Plan, aber so klappt es nun:

<?php

declare(strict_types=1);

/*
 * This file is part of the ContaoIpLoginBundle.
 *
 * (c) inspiredminds
 *
 * @license LGPL-3.0-or-later
 */

namespace InspiredMinds\ContaoIpLoginBundle\Security;

use Contao\CoreBundle\Framework\ContaoFramework;
use Contao\Date;
use Contao\MemberModel;
use Contao\StringUtil;
use Contao\System;
use Symfony\Component\ExpressionLanguage\ExpressionLanguage;
use Symfony\Component\HttpFoundation\IpUtils;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Security\Core\Authentication\Token\TokenInterface;
use Symfony\Component\Security\Core\Exception\AuthenticationException;
use Symfony\Component\Security\Core\Security;
use Symfony\Component\Security\Core\User\UserInterface;
use Symfony\Component\Security\Core\User\UserProviderInterface;
use Symfony\Component\Security\Guard\AbstractGuardAuthenticator;
use Symfony\Component\HttpFoundation\RedirectResponse;

class IpAuthenticator extends AbstractGuardAuthenticator
{
    /** @var Security */
    private $security;

    /** @var ContaoFramework */
    private $framework;

    /** @var string|null */
    private $requestCondition;

    public function __construct(Security $security, ContaoFramework $framework, array $allowedIps, array $ignoredPaths, ?string $requestCondition)
    {
        $this->security = $security;
        $this->framework = $framework;
        $this->requestCondition = $requestCondition;
    }

    public function supports(Request $request): bool
    {
        if($request->request->get('FORM_SUBMIT')  == 'tl_login_125'){
            return true;
        }
        return false;
    }

    public function getCredentials(Request $request): string
    {
        if($request->request->get('FORM_SUBMIT')  == 'tl_login_125'){
            return $request->request->get('username');
        }
        return '';
    }

    public function getUser($credentials, UserProviderInterface $userProvider): ?UserInterface
    {
        $request = System::getContainer()->get('request_stack')->getCurrentRequest();
        if($request->request->get('FORM_SUBMIT')  == 'tl_login_125'){
            return $userProvider->loadUserByUsername($request->request->get('username'));
        }
        return null;
    }

    public function checkCredentials($credentials, UserInterface $user): bool
    {
        $request = System::getContainer()->get('request_stack')->getCurrentRequest();
        
        // LDAP-Elemente zur Authentifizierung
        $ldaprdn  = 'CN='.$request->request->get('username');
        $ldappass = $request->request->get('password');
        // Verbinden mit dem LDAP-Server
        $ldapconn = ldap_connect("ldap://xxx.de")
            or die("Keine Verbindung zum LDAP-Server möglich.");

        if ($ldapconn) {
            // Anmelden am LDAP-Server (als Member)
            $ldapbind = ldap_bind($ldapconn, $ldaprdn, $ldappass);
            // Überprüfung der Authentifizierung
            if ($ldapbind && $request->request->get('password') != ''){
                return true;
            } else {
                return false;
            }
        }
    }

    public function onAuthenticationSuccess(Request $request, TokenInterface $token, $providerKey): ?Response
    {
        return null;
    }

    public function onAuthenticationFailure(Request $request, AuthenticationException $exception): ?Response
    {
        if($request->request->get('FORM_SUBMIT')  == 'tl_login_125'){
            $request = System::getContainer()->get('request_stack')->getCurrentRequest();
            System::log('Test 8 Fail'.print_r($request->getPathInfo(),true), __METHOD__, TL_ACCESS);
            $request->getSession()->set(Security::AUTHENTICATION_ERROR, $exception);
            return new RedirectResponse($request->headers->get('referer'));
        }
        return null;
    }

    public function supportsRememberMe(): bool
    {
        return false;
    }

    public function start(Request $request, AuthenticationException $authException = null): ?Response
    {
        return null;
    }
}

Bin offen für Verbesserung oder andere Lösungen.

[d.tafel]

Ich habe es aber auf brechen und biegen nicht hinbekommen das die Security im src genommen wird. Daher der Umbau einer Erweiterung mit Security Datei...