Skip to content

Commit

Permalink
feature #26832 [MonologBridge] Added WebSubscriberProcessor to ease p…
Browse files Browse the repository at this point in the history
…rocessor configuration (lyrixx)

This PR was merged into the 4.1-dev branch.

Discussion
----------

[MonologBridge] Added WebSubscriberProcessor to ease processor configuration

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets |
| License       | MIT
| Doc PR        |

---

Side note: I could be nice to have an interface to act as the marker to AutoConfigure processors

Commits
-------

b1287d6 [MonologBridge] Added WebSubscriberProcessor to ease processor configuration
  • Loading branch information
fabpot committed Apr 23, 2018
2 parents 2ed0c67 + b1287d6 commit 9ae116f
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 1 deletion.
5 changes: 5 additions & 0 deletions src/Symfony/Bridge/Monolog/CHANGELOG.md
@@ -1,6 +1,11 @@
CHANGELOG
=========

4.1.0
-----

* `WebProcessor` now implements `EventSubscriberInterface` in order to be easily autoconfigured

4.0.0
-----

Expand Down
11 changes: 10 additions & 1 deletion src/Symfony/Bridge/Monolog/Processor/WebProcessor.php
Expand Up @@ -12,14 +12,16 @@
namespace Symfony\Bridge\Monolog\Processor;

use Monolog\Processor\WebProcessor as BaseWebProcessor;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\HttpKernel\Event\GetResponseEvent;
use Symfony\Component\HttpKernel\KernelEvents;

/**
* WebProcessor override to read from the HttpFoundation's Request.
*
* @author Jordi Boggiano <j.boggiano@seld.be>
*/
class WebProcessor extends BaseWebProcessor
class WebProcessor extends BaseWebProcessor implements EventSubscriberInterface
{
public function __construct(array $extraFields = null)
{
Expand All @@ -34,4 +36,11 @@ public function onKernelRequest(GetResponseEvent $event)
$this->serverData['REMOTE_ADDR'] = $event->getRequest()->getClientIp();
}
}

public static function getSubscribedEvents()
{
return array(
KernelEvents::REQUEST => array('onKernelRequest', 4096),
);
}
}

0 comments on commit 9ae116f

Please sign in to comment.