search

quba / ZPI

System obsługujący konferencję naukową
MIT License
3 stars 1 forks source link

Redirect przy próbie dostępu do zasobów wymagających logowania #8

Closed quba closed 13 years ago

quba commented 13 years ago

Wiem na czym polega problem z redirectem. Spójrzcie tutaj na funkcję resetLocale(): https://github.com/schmittjoh/symfony/commit/f300edebe4c27171a4fc7c6dfd0a71f219a6d850#diff-9 Chodzi o plik HttpUtils. W tym wątku jest napisane, że któryś dev dał na podobny problem przy locale taki wlasnie path: https://github.com/symfony/symfony/issues/1493#issuecomment-1590182 Ten cały schmittjoh napisal tego patha, ktory jest w linku pierwszym. Problem się rozwiązuje, gdy do tej funkcji resetLocale dodamy: if (isset($parameters['_conf'])) { $context->setParameter('_conf', $parameters['_conf']); } Ale niestety nie jest najlepszą praktyką ingerowanie w kod frameworka. Ktoś ma pomysł jak to inaczej zrobić?

Przypisane do @lyzkova, jako że to jego świetny pomysł z obsługą wielu konferencji. I tak już za dużo zrobiłem w tej kwestii.

lyzkov commented 13 years ago

Szkoda, że nie podałeś przykładu gdzie trzeba wejść (url) żeby zobaczyć jak wygląda ten issue w praniu. :)

quba commented 13 years ago

akcja zalogowania, akcja wylogowania, akcja wejscia na podstrone wymagajaca zalogowania (ten redirect do login forma).

lyzkov commented 13 years ago

Z tego co pamiętam to do tej pory nie mam jeszcze w moich kontrolerach autoryzacji. Tak więc można do nich wejść nawet będąc niezalogowanym.

quba commented 13 years ago

No a powinna być.

Rozwiązałem sam tego issue'a poprzez podmianę klasy HttpUtils na własną. Zaraz pójdzie commit z poprawką. Zamykam.