[FrameworkBundle] Extends the RequestDataCollector
Moves the logic of collect controller data while forwarding in the framework bundle.
This commit is contained in:
parent
227ac77f11
commit
4f020b5686
@ -0,0 +1,77 @@
|
|||||||
|
<?php
|
||||||
|
|
||||||
|
/*
|
||||||
|
* This file is part of the Symfony package.
|
||||||
|
*
|
||||||
|
* (c) Fabien Potencier <fabien@symfony.com>
|
||||||
|
*
|
||||||
|
* For the full copyright and license information, please view the LICENSE
|
||||||
|
* file that was distributed with this source code.
|
||||||
|
*/
|
||||||
|
|
||||||
|
namespace Symfony\Bundle\FrameworkBundle\DataCollector;
|
||||||
|
|
||||||
|
use Symfony\Component\HttpFoundation\ParameterBag;
|
||||||
|
use Symfony\Component\HttpFoundation\Request;
|
||||||
|
use Symfony\Component\HttpFoundation\Response;
|
||||||
|
use Symfony\Component\HttpKernel\DataCollector\RequestDataCollector as BaseRequestCollector;
|
||||||
|
use Symfony\Component\HttpKernel\Event\FilterControllerEvent;
|
||||||
|
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* RequestDataCollector.
|
||||||
|
*
|
||||||
|
* @author Jules Pietri <jusles@heahprod.com>
|
||||||
|
*/
|
||||||
|
class RequestDataCollector extends BaseRequestCollector implements EventSubscriberInterface
|
||||||
|
{
|
||||||
|
/**
|
||||||
|
* {@inheritdoc}
|
||||||
|
*/
|
||||||
|
public function collect(Request $request, Response $response, \Exception $exception = null)
|
||||||
|
{
|
||||||
|
parent::collect($request, $response, $exception);
|
||||||
|
|
||||||
|
if ($parentRequestAttributes = $request->attributes->get('_forwarded')) {
|
||||||
|
if ($parentRequestAttributes instanceof ParameterBag) {
|
||||||
|
$parentRequestAttributes->set('_forward_token', $response->headers->get('x-debug-token'));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
if ($request->attributes->has('_forward_controller')) {
|
||||||
|
$this->data['forward'] = array(
|
||||||
|
'token' => $request->attributes->get('_forward_token'),
|
||||||
|
'controller' => $this->parseController($request->attributes->get('_forward_controller')),
|
||||||
|
);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* Gets the parsed forward controller.
|
||||||
|
*
|
||||||
|
* @return array|bool An array with keys 'token' the forward profile token, and
|
||||||
|
* 'controller' the parsed forward controller, false otherwise
|
||||||
|
*/
|
||||||
|
public function getForward()
|
||||||
|
{
|
||||||
|
return isset($this->data['forward']) ? $this->data['forward'] : false;
|
||||||
|
}
|
||||||
|
|
||||||
|
public function onKernelController(FilterControllerEvent $event)
|
||||||
|
{
|
||||||
|
$this->controllers[$event->getRequest()] = $event->getController();
|
||||||
|
|
||||||
|
if ($parentRequestAttributes = $event->getRequest()->attributes->get('_forwarded')) {
|
||||||
|
if ($parentRequestAttributes instanceof ParameterBag) {
|
||||||
|
$parentRequestAttributes->set('_forward_controller', $event->getController());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* {@inheritdoc}
|
||||||
|
*/
|
||||||
|
public function getName()
|
||||||
|
{
|
||||||
|
return 'request';
|
||||||
|
}
|
||||||
|
}
|
@ -10,7 +10,7 @@
|
|||||||
<call method="setKernel"><argument type="service" id="kernel" on-invalid="ignore" /></call>
|
<call method="setKernel"><argument type="service" id="kernel" on-invalid="ignore" /></call>
|
||||||
</service>
|
</service>
|
||||||
|
|
||||||
<service id="data_collector.request" class="Symfony\Component\HttpKernel\DataCollector\RequestDataCollector">
|
<service id="data_collector.request" class="Symfony\Bundle\FrameworkBundle\DataCollector\RequestDataCollector">
|
||||||
<tag name="kernel.event_subscriber" />
|
<tag name="kernel.event_subscriber" />
|
||||||
<tag name="data_collector" template="@WebProfiler/Collector/request.html.twig" id="request" priority="335" />
|
<tag name="data_collector" template="@WebProfiler/Collector/request.html.twig" id="request" priority="335" />
|
||||||
</service>
|
</service>
|
||||||
|
@ -27,6 +27,7 @@ use Symfony\Component\EventDispatcher\EventSubscriberInterface;
|
|||||||
*/
|
*/
|
||||||
class RequestDataCollector extends DataCollector implements EventSubscriberInterface
|
class RequestDataCollector extends DataCollector implements EventSubscriberInterface
|
||||||
{
|
{
|
||||||
|
/** @var \SplObjectStorage */
|
||||||
protected $controllers;
|
protected $controllers;
|
||||||
|
|
||||||
public function __construct()
|
public function __construct()
|
||||||
@ -286,26 +287,9 @@ class RequestDataCollector extends DataCollector implements EventSubscriberInter
|
|||||||
return isset($this->data['redirect']) ? $this->data['redirect'] : false;
|
return isset($this->data['redirect']) ? $this->data['redirect'] : false;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* Gets the parsed forward controller.
|
|
||||||
*
|
|
||||||
* @return array|bool An array with keys 'token' the forward profile token, and
|
|
||||||
* 'controller' the parsed forward controller, false otherwise
|
|
||||||
*/
|
|
||||||
public function getForward()
|
|
||||||
{
|
|
||||||
return isset($this->data['forward']) ? $this->data['forward'] : false;
|
|
||||||
}
|
|
||||||
|
|
||||||
public function onKernelController(FilterControllerEvent $event)
|
public function onKernelController(FilterControllerEvent $event)
|
||||||
{
|
{
|
||||||
$this->controllers[$event->getRequest()] = $event->getController();
|
$this->controllers[$event->getRequest()] = $event->getController();
|
||||||
|
|
||||||
if ($parentRequestAttributes = $event->getRequest()->attributes->get('_forwarded')) {
|
|
||||||
if ($parentRequestAttributes instanceof ParameterBag) {
|
|
||||||
$parentRequestAttributes->set('_forward_controller', $event->getController());
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
public static function getSubscribedEvents()
|
public static function getSubscribedEvents()
|
||||||
@ -321,43 +305,6 @@ class RequestDataCollector extends DataCollector implements EventSubscriberInter
|
|||||||
return 'request';
|
return 'request';
|
||||||
}
|
}
|
||||||
|
|
||||||
private function getCookieHeader($name, $value, $expires, $path, $domain, $secure, $httponly)
|
|
||||||
{
|
|
||||||
$cookie = sprintf('%s=%s', $name, urlencode($value));
|
|
||||||
|
|
||||||
if (0 !== $expires) {
|
|
||||||
if (is_numeric($expires)) {
|
|
||||||
$expires = (int) $expires;
|
|
||||||
} elseif ($expires instanceof \DateTime) {
|
|
||||||
$expires = $expires->getTimestamp();
|
|
||||||
} else {
|
|
||||||
$tmp = strtotime($expires);
|
|
||||||
if (false === $tmp || -1 == $tmp) {
|
|
||||||
throw new \InvalidArgumentException(sprintf('The "expires" cookie parameter is not valid (%s).', $expires));
|
|
||||||
}
|
|
||||||
$expires = $tmp;
|
|
||||||
}
|
|
||||||
|
|
||||||
$cookie .= '; expires='.str_replace('+0000', '', \DateTime::createFromFormat('U', $expires, new \DateTimeZone('GMT'))->format('D, d-M-Y H:i:s T'));
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($domain) {
|
|
||||||
$cookie .= '; domain='.$domain;
|
|
||||||
}
|
|
||||||
|
|
||||||
$cookie .= '; path='.$path;
|
|
||||||
|
|
||||||
if ($secure) {
|
|
||||||
$cookie .= '; secure';
|
|
||||||
}
|
|
||||||
|
|
||||||
if ($httponly) {
|
|
||||||
$cookie .= '; httponly';
|
|
||||||
}
|
|
||||||
|
|
||||||
return $cookie;
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Parse a controller.
|
* Parse a controller.
|
||||||
*
|
*
|
||||||
@ -365,7 +312,7 @@ class RequestDataCollector extends DataCollector implements EventSubscriberInter
|
|||||||
*
|
*
|
||||||
* @return array|string An array of controller data or a simple string
|
* @return array|string An array of controller data or a simple string
|
||||||
*/
|
*/
|
||||||
private function parseController($controller)
|
protected function parseController($controller)
|
||||||
{
|
{
|
||||||
if (is_string($controller) && false !== strpos($controller, '::')) {
|
if (is_string($controller) && false !== strpos($controller, '::')) {
|
||||||
$controller = explode('::', $controller);
|
$controller = explode('::', $controller);
|
||||||
@ -418,4 +365,41 @@ class RequestDataCollector extends DataCollector implements EventSubscriberInter
|
|||||||
|
|
||||||
return (string) $controller ?: 'n/a';
|
return (string) $controller ?: 'n/a';
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private function getCookieHeader($name, $value, $expires, $path, $domain, $secure, $httponly)
|
||||||
|
{
|
||||||
|
$cookie = sprintf('%s=%s', $name, urlencode($value));
|
||||||
|
|
||||||
|
if (0 !== $expires) {
|
||||||
|
if (is_numeric($expires)) {
|
||||||
|
$expires = (int) $expires;
|
||||||
|
} elseif ($expires instanceof \DateTime) {
|
||||||
|
$expires = $expires->getTimestamp();
|
||||||
|
} else {
|
||||||
|
$tmp = strtotime($expires);
|
||||||
|
if (false === $tmp || -1 == $tmp) {
|
||||||
|
throw new \InvalidArgumentException(sprintf('The "expires" cookie parameter is not valid (%s).', $expires));
|
||||||
|
}
|
||||||
|
$expires = $tmp;
|
||||||
|
}
|
||||||
|
|
||||||
|
$cookie .= '; expires='.str_replace('+0000', '', \DateTime::createFromFormat('U', $expires, new \DateTimeZone('GMT'))->format('D, d-M-Y H:i:s T'));
|
||||||
|
}
|
||||||
|
|
||||||
|
if ($domain) {
|
||||||
|
$cookie .= '; domain='.$domain;
|
||||||
|
}
|
||||||
|
|
||||||
|
$cookie .= '; path='.$path;
|
||||||
|
|
||||||
|
if ($secure) {
|
||||||
|
$cookie .= '; secure';
|
||||||
|
}
|
||||||
|
|
||||||
|
if ($httponly) {
|
||||||
|
$cookie .= '; httponly';
|
||||||
|
}
|
||||||
|
|
||||||
|
return $cookie;
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
Reference in New Issue
Block a user