Nette – authenticator …

vojtech.dobes napsal(a):

Co sa tyka konfiguracie, mam to nastavene takto, a nefunguje

	services:
		database: @Nette\Database\Connection

		authenticator: MyAuthenticator( @database::table(users) )

takze ak mas prosim inu radu, sem s nou :-) … a co sa tyka toho mojho skrateneho textu … ja osm mal pocit ze som ho nacrtol jasne … no mozno nie (ale to som potom nemal v plane)

Mam to nastavene, ako je napisane vyssie, ale meno a heslo, ktore mam v databaze mi neakceptuje …

A ked ten MyAuthenticator zaregistrujem …$user->setAuthenticator(new MyAuthenticator); ukazuje mi nasledovnu chybu

Argument 1 passed to MyAuthenticator::__construct() must be an instance of Nette\Database\Connection, none given, called in C:\Program Files (x86)\EasyPHP-5.3.8.0\www\PUSKAS\app\presenters\LoginPresenter.php on line 20 and defined

ricco24 napsal(a):

1. Pouzitim registracie: $user->setAuthenticator($this->context->authenticator); mi hlasi nasledovnu chybu

Argument 1 passed to MyAuthenticator::__construct() must be an instance of Nette\Database\Connection, instance of Nette\Database\Table\Selection given, called in C:\Program Files (x86)\EasyPHP-5.3.8.0\www\PUSKAS\temp\cache\_Nette.Configurator\_-285bac8f7b7d3ea72485a759a142673d.php on line 108 and defined search►

a pre ISTOTU este raz ukazem moj Presenter … (trosicka zmeneny)

<?php

class LoginPresenter extends BasePresenter
{
    public function renderDefault()
    {
        $user = $this->getUser();

        if($user->isLoggedIn()){
            $this->redirect('Homepage:default');
        }

        $user->setAuthenticator($this->context->authenticator);

        if ($this['login']->isSuccess()){
            $username = $_POST['username'];
            $password = $_POST['password'];
        /*Pokusa sa prihlasit*/
            try {
                // pokusíme se přihlásit uživatele...
                $user->login($username, $password);
            $this->redirect('Homepage:default');
                // ...a v případě úspěchu presměrujeme na další stránku
                //$this->redirect('Homepage:default');
            } catch (Nette\Security\AuthenticationException $e) {
                $e->getMessage();
            }
        }
    }

    public function createComponentLogin(){
        $form = new Nette\Application\UI\Form;
        $form->setAction('');
        $form->setMethod('POST');
        $form->addText('username',null,16);
        $form->addPassword('password',null,16);
        $form->addSubmit('login','Login');

        return $form;
    }
}
?>

2. MyAuthenticator.php mam vytvorenu:
Nazov_Projektu
|
|-app
|-models
|-MyAuthenticator.php

3. Konfiguracny dubor je nacrtnuty vyssie

spatne registrujes authenticator v configu (predavas mu spatnou sluzbu). nevidim duvod proc nastavovat auth pres setAuthenticator … o to se ti postara autowiring

ricco24 napsal(a):

Tak tu je ta trieda … a zmenil som Nette\Database\Connection na Nette\Database\Table\Selection ako si mi poradil .. no stale chyba

<?php

use Nette\Security as NS;

class MyAuthenticator extends Nette\Object implements NS\IAuthenticator
{
    public $connection;

    function __construct(Nette\Database\Table\Selection $connection)
    {
        $this->connection = $connection;
    }

    function authenticate(array $credentials)
    {
        list($username, $password) = $credentials;
        $row = $this->connection->table('users')
            ->where('username', $username)->fetch();

        if (!$row) {
            throw new NS\AuthenticationException('User not found.');
        }

        if ($row->password !== md5($password)) {
            throw new NS\AuthenticationException('Invalid password.');
        }

        return new NS\Identity($row->id, $row->role);
    }
}
?>

A co sa tyka config.neon, tak tam som to takisto zmenil tak ako mi to poradil ricco24

.. stale ma problem v triede MyAuthenticator vo funkcii __construct()

… zmen to zpatky na Connection a ten cfg nech jak je

Jenom takový „detail“, ta tvá logika by měla být spíše v action fázi, nikoliv až v render.

Jenom takový „detail“, ta tvá logika by měla být spíše v action fázi, nikoliv až v render.

ona by spis mela bejt v nakym callbacku navazanym na onSuccess

davidm napsal(a):
ona by spis mela bejt v nakym callbacku navazanym na onSuccess

Souhlasím a nechápu, proč dotyčný píše:

$username = $_POST['username'];
$password = $_POST['password'];

Doporučuju ti vypsat hodnoty, které se pošlou z formuláře a ověřit, zda se opravdu shodují s hodnotami v db.

Editoval JuniorJR (22. 4. 2012 21:50)

Zdá sa mi že v tom je už príliž velký zmätok. Toto riešenie by ti malo fungovať. Inak radím ti pozrieť si aj SignPresenter zo sandboxu. Je to skoro to isté.

class LoginPresenter extends BasePresenter
{
    public function renderDefault()
    {
        $user = $this->getUser();

        if($user->isLoggedIn()){
            $this->redirect('Homepage:default');
        }
    }

   public function createComponentLoginForm(){

        $form = new Nette\Application\UI\Form;
        $form->addText('username',null,16);
        $form->addPassword('password',null,16);
        $form->addSubmit('login','Login');

	$form->onSuccess[] = callback($this, 'loginFormSubmitted')

        return $form;
   }


   public function loginFormSubmitted($form) {
	$values = $form->getValues();

        try {
                $user->login($values->username, $values->password);
                $this->redirect('Homepage:default');
            } catch (Nette\Security\AuthenticationException $e) {
                $e->getMessage();
            }
   }
}

Authenticator použijeme pre jednoduchosť ten ktorý je uvedený v dokumentácii a ktorý už máš implementovaný

use Nette\Security as NS;

class MyAuthenticator extends Nette\Object implements NS\IAuthenticator
{
    public $connection;

    function __construct(Nette\Database\Connection $connection)
    {
        $this->connection = $connection;
    }

    function authenticate(array $credentials)
    {
        list($username, $password) = $credentials;
        $row = $this->connection->table('users')
            ->where('username', $username)->fetch();

        if (!$row) {
            throw new NS\AuthenticationException('User not found.');
        }

        if ($row->password !== md5($password)) {
            throw new NS\AuthenticationException('Invalid password.');
        }

        return new NS\Identity($row->id, $row->role);
    }
}

a v config.neon

services:
        database: @Nette\Database\Connection

        authenticator: MyAuthenticator( @database )

Dúfam že som na nič nezabudol.

Editoval ricco24 (22. 4. 2012 22:19)

@ricco24: toto je co if ($form->isSuccess())? nejsi náhodou v callbacku onSuccess[], tedy v callbacku, který se vykoná je v případě úspěšného odeslání?

@takeshi: v distribuci máš tvůj problém vyřešený, jmenuje se to sandbox, tam si prohlídni, ja se to správně řeší.

SimpleAuthenticator v neonu:

nette:
	security:
		users:
			admin: admin123

Editoval 22 (22. 4. 2012 22:17)

No tak ja som v koncoch … chybu mi neukazuje, prihlasit sa mi aj tak neda (zmenil som config, loginPresenter, MyAuthenticator) … tak ako to urobil ricco24 a nic.

Co sa tyka SimpleAuthenticator … pomocou neho mi to islo, len som pri tom nepouyival databazu

Ak by pomohlo poslal by som aj cely moj mini-prijekt … na ktorom si skusam iba nove veci

Můžu se zeptat, proč nepoužiješ ten sandbox? Tam je přihlašování už udělané a 100% fungující…

22 napsal(a):

Můžu se zeptat, proč nepoužiješ ten sandbox? Tam je přihlašování už udělané a 100% fungující…

Kde prosim ta najdem ten SANDBOX? som ho hladal, ale nenasiel.

Diki moc :-) … P.S skus si precitat moju predoslu spravu, ci tam nenajdes uz iba drobnu chybu

Myslím, že Ot@s bude mít pravdu, vydumpuj si pomocí dump(), jake heslo se posílá na ověření a jaké heslo je v DB, pokud se neshodují, tak samozřejmě to přes Authenticator nemůže projít.

P.S.: Sandbox je přímo v dustribuci Nette, jeden z adresářu, kde je předpřipavená kostra aplikace a je tam i SignPresenter s přihlašovacím formulářem včetně modelu a Authenticatoru.

Editoval 22 (22. 4. 2012 23:06)

UZ som nasiel chybu, ⇒ uz sa mi da prihlasit (pouzil som Authenticator nie MyAuth…)

CHYBA BOLA V tomto

<?php
if ($row->password !== $this->calculateHash($password)) {...}
?>

, ktory ked som zmenil na

<?php
if ($row->password !== $password) {...}
?>

tak to islo … problem pravdepodobne ale teraz nastal, ze to nebude tak bezpecne

co na to hovorite??

Myiyk napsal(a):

O dost bezpečnější je ukládat heslo jako hash.
Ale jestli to spouštíš jen sám sobě na localhostu, tak je to jedno.

Ja viem, ze to je menej bezpecne, ked zamenim toto

<?php
if ($row->password !== $this->calculateHash($password)) {...}
?>

za toto …

<?php
if ($row->password !== $password) {...}
?>

… ale ked pouzijem hash, tak mam problem z heslom, teda nezhoduje sa … len v tom druhom pripade, bez calculateHash

A aby sa nepovedalo, skusal som aj prihlasovanie z prikladu, zo sandboxu, jednoducho som si skopiroval subory signPresenter a jeho sablonu, ale bez vysledne (fungovalo to iba ak som pouzil SQLITE databazu ‚demo.db‘)

Editoval Takeshi (23. 4. 2012 22:47)

duke napsal(a):

Takeshi napsal(a):
… ale ked pouzijem hash, tak mam problem z heslom, teda nezhoduje sa … len v tom druhom pripade, bez calculateHash

Aby ti to fungovalo, musíš si do db to heslo ukládat zahashované. A pokud ti jde o bezpečnost, tak bys měl také solit. :-)

No vidis, to som nevedel … clovek sa uci kazdym dnom :-)