Záhada s přihlášením jiného uživatele, přepsání identity, UserPanel

Tak a aby toho nebylo málo, dumpnul jsem pro jistotu i

$this->getUser()->getStorage()

a její private proměnná sessionSection obsahuje pod klíčem

["Nette.Http.UserStorage/"]["identity"]

správně identitu uživatele Dva. Jak je možné že SessionSection obsahuje data, která nejsou v $_SESSION? Připadám si trošinku jako idiot. :-)

Editoval elendir (23. 10. 2012 17:21)

Pomůže když předtím ->login() explicitně zavoláš ->logout()?

Tak jsem se díval na tu metodu logout a obávám se že její explicitní volání nepomůže, poněvadž hned první řádek ve fci User::login() je

$this->logout(TRUE);

Popíšu zde podrobněji co jsem objevil, kritické části kódu jsou podle mně: metoda start v SessionSection, kde dojde k „napojení“ dvou parametrů třídy do pole $_SESSION by-reference:

$this->data = & $_SESSION['__NF']['DATA'][$this->name];
$this->meta = & $_SESSION['__NF']['META'][$this->name];

a následně metoda regenerateId ve třídě Session kde dojde ke zkopírování pole $_SESSION, což jestli se nepletu rozbije tyto reference, respektive tyto reference poté vedou do neaktuálního pole $_SESSION.

Proces při zavolání User::login() je následující:

1. User::logout() odhlašuje aktuálně přihlášeného, to vede k volání Session::regenerateId() a dochází k tomu, že od tohoto okamžiku již výše uvedené reference odkazují do neaktuálního $_SESSION pole a všechna data zápsaná do session prostřednictvím třídy SessionSection od tohoto momentu dál budou tím pádem ztracena.
2. User::login() se provede v pořádku ale identita se zapíše do neaktuálního $_SESSION pole.
3. Po úspěšném přihlášení dochází k přesměrování ale nová identita není v $_SESSION, byla totiž uložena jinam.

Je sice fakt pozdě takže jsem možná úplně mimo a možná je to celé blbost, ale strávil jsem nad tím 15 hodin, velmi prosím nějakého zkušeného nettistu jestli by se na to mohl mrknout a ověřit to. Nebude-li mé zoufalé volání vyslyšeno zde, šoupnu to do hlášení chyb.

Editoval elendir (23. 10. 2012 23:39)

Ještě bych chtěl dodat, že zde pozoruji určitou souvislost s tímto poměrně ujetým problémem, který popisoval @mcmatak. Ale teď už jdu fakt spát.

Tak abych to uzavřel: hodil jsem to do issues na Githubu. Opravit se mi to nepodařilo a nejsem schopen tomu v této chvíli věnovat více času. Byl jsem proto nucen zakomentovat problematické řádky v Nette\Http\Session::regenerateId(). Vzhledem k tomu, že se jedná o základ práce se session na kterém je závislá hromada věcí, těžko říct co všechno tím může být ještě ovlivněné. Doporučoval bych proto, aby to někdo (šikovnější než já) opravil.

Potvrzuji že problém přetrvává po vymazání cookies a je nezávislý na prohlížeči (testováno Chrome, Opera, FF). Po zakomentování těchto řádek ve fci regenerateId

session_regenerate_id(TRUE);
session_write_close();
$backup = $_SESSION;
session_start();
$_SESSION = $backup;

je problém odstraněn. Původně (ještě před tím než jsem to zbaběle zakomentoval) jsem se tam pokusil udržet ty 2 výše popisované reference pomocí

$backup = & $_SESSION;
session_start();
$_SESSION = & $backup;

ale to nepomohlo. Tipnul bych si že session_start() to pole $_SESSION vyNULLuje nebo s ním prostě provede nějakou operaci která breakne ty 2 reference vedoucí dovnitř $_SESSION[‚__NF‘][‚DATA‘], ale to je fakt jen nápad, neměl jsem bohužel čas to ověřit.

EDIT: Jo a ve které verzi se chyba objevila bohužel také nevím, ale s největší pravděpodobností v okamžiku kdy se objevil ten výše uvedný blok kódu, což bylo podle Githubu nějak cca před 7 měsíci jestli si dobře vzpomínám. Moje původní verze nette byla určitě stará alespoň rok, tzn tento kód neobsahovala a popisovaný problém neexistoval.

EDIT2: Ještě bych chtěl zdůraznit hlavní vodítko – problém se projevuje pouze když se pokusím zalogovat jako uživatel DVA v okamžiku kdy jsem zalogován jako uživatel JEDNA. V případě, že se nejprve pomocí Sign:out odloguju, jsem přesměrován na Sign:in a zaloguji se jako užiovatel DVA, proběhne vše v pořádku.

EDIT3: Opcode cache nemám. I nette cache promazávám. Tohle už je snad poslední edit, sorry. :-)

Editoval elendir (25. 10. 2012 11:36)

Pod jakou verzí PHP se tak děje?

session_start() skutečně pole $_SESSION zruší a znovuvytvoří, právě proto je tam použitá proměnná $backup. Takže reference se nebreaknou a stále vedou do pole $_SESSION.

David Grudl napsal(a):

session_start() skutečně pole $_SESSION zruší a znovuvytvoří, právě proto je tam použitá proměnná $backup. Takže reference se nebreaknou a stále vedou do pole $_SESSION.

Špatně jsem se vyjádřil, měl jsem na mysli alternativu, kdy session_start() provede něco přímo s obsahem $_SESSION, tj. že si sáhne na ty reference a nějak pokazí přímo tu hodnotu na kterou tyto reference směřují (čímž pokazí i $backup). Taky to může dělat už ta fce session_write_close(). Fakt mě mrzí že jsem neměl víc času to dohledat, sorry.

EDIT: Možná že plácám blbosti, už jsem to půl roku neviděl.
EDIT2: Jasně, plácám blbosti, zapomněl jsem že ta reference směřuje do $_SESSION a nikoliv ze $_SESSION ven, takže tenhle post prosím ignoruj.

Editoval elendir (16. 4. 2013 16:39)

Mohl by jsi mi poslat emailem nějaké torzo aplikace, kde se to projevuje, s postupem, jak chyby docilit?

Určitě pošli!