Skip to content

Conversation

iteman
Copy link
Member

@iteman iteman commented Sep 30, 2015

This problem is due to the order of the listeners for kernel.request as the following:

Order Callable
1 Symfony\Component\HttpKernel\EventListener\DebugHandlersListener::configure()
2 Symfony\Bundle\FrameworkBundle\EventListener\SessionListener::onKernelRequest()
3 Symfony\Component\HttpKernel\EventListener\FragmentListener::onKernelRequest()
4 Symfony\Component\HttpKernel\EventListener\RouterListener::onKernelRequest()
5 Symfony\Component\HttpKernel\EventListener\LocaleListener::onKernelRequest()
6 Symfony\Component\HttpKernel\EventListener\TranslatorListener::onKernelRequest()
7 Symfony\Component\Security\Http\Firewall::onKernelRequest()
8 PHPMentors\ProxyURLRewriteBundle\EventListener\ProxyUrlRewriteListener::onKernelRequest()

This PR changes the order as the following:

Order Callable
1 Symfony\Component\HttpKernel\EventListener\DebugHandlersListener::configure()
2 Symfony\Bundle\FrameworkBundle\EventListener\SessionListener::onKernelRequest()
3 Symfony\Component\HttpKernel\EventListener\FragmentListener::onKernelRequest()
4 Symfony\Component\HttpKernel\EventListener\RouterListener::onKernelRequest()
5 PHPMentors\ProxyURLRewriteBundle\EventListener\ProxyUrlRewriteListener::onKernelRequest()
6 Symfony\Component\HttpKernel\EventListener\LocaleListener::onKernelRequest()
7 Symfony\Component\HttpKernel\EventListener\TranslatorListener::onKernelRequest()
8 Symfony\Component\Security\Http\Firewall::onKernelRequest()

@iteman iteman added the bug label Sep 30, 2015
@iteman iteman self-assigned this Sep 30, 2015
@iteman iteman added this to the 1.1.1 milestone Sep 30, 2015
iteman added a commit that referenced this pull request Sep 30, 2015
…rity-fix

Urls are not rewritten on redirection from authentication
@iteman iteman merged commit 8373b73 into phpmentors-jp:1.1 Sep 30, 2015
@iteman iteman deleted the unexpected-listener-order-with-security-fix branch September 30, 2015 03:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant