Skip to content

Commit

Permalink
[Security] fixed sub-requests creation (closes #1212)
Browse files Browse the repository at this point in the history
  • Loading branch information
fabpot committed Jun 8, 2011
1 parent 9118fce commit 188e742
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 3 deletions.
Expand Up @@ -50,7 +50,9 @@ public function start(Request $request, AuthenticationException $authException =
{
$path = str_replace('{_locale}', $request->getSession()->getLocale(), $this->loginPath);
if ($this->useForward) {
return $this->httpKernel->handle(Request::create($path), HttpKernelInterface::SUB_REQUEST);
$subRequest = Request::create($path, 'get', array(), $request->cookies->all(), array(), $request->server->all());

return $this->httpKernel->handle($subRequest, HttpKernelInterface::SUB_REQUEST);
}

return new RedirectResponse(0 !== strpos($path, 'http') ? $request->getUriForPath($path) : $path, 302);
Expand Down
Expand Up @@ -203,7 +203,7 @@ private function onFailure(GetResponseEvent $event, Request $request, Authentica
$this->logger->debug(sprintf('Forwarding to %s', $path));
}

$subRequest = Request::create($path);
$subRequest = Request::create($path, 'get', array(), $request->cookies->all(), array(), $request->server->all());
$subRequest->attributes->set(SecurityContextInterface::AUTHENTICATION_ERROR, $failed);

return $event->getKernel()->handle($subRequest, HttpKernelInterface::SUB_REQUEST);
Expand Down
Expand Up @@ -115,7 +115,7 @@ public function onCoreException(GetResponseForExceptionEvent $event)
return;
}

$subRequest = Request::create($this->errorPage);
$subRequest = Request::create($this->errorPage, 'get', array(), $request->cookies->all(), array(), $request->server->all());
$subRequest->attributes->set(SecurityContextInterface::ACCESS_DENIED_ERROR, $exception);

$response = $event->getKernel()->handle($subRequest, HttpKernelInterface::SUB_REQUEST, true);
Expand Down

0 comments on commit 188e742

Please sign in to comment.