Merge pull request #6 from aptoma/catch-subnet-ip-error

gunnarlium · gunnarlium · commit f6646c61ba08 · 2016-02-10T14:26:28.000+01:00
Catch error when trying to retrieve client IP for logging purposes
diff --git a/src/Aptoma/Log/RequestProcessor.php b/src/Aptoma/Log/RequestProcessor.php
@@ -32,7 +32,11 @@ public function __construct(Application $app, $token = null)
 
     public function __invoke(array $record)
     {
-        $record['extra']['clientIp'] = $this->getClientIp($this->app['request_stack']);
+        try {
+            $record['extra']['clientIp'] = $this->getClientIp($this->app['request_stack']);
+        } catch (\Exception $e) {
+            $record['extra']['clientIp'] = '';
+        }
         if ($this->app->offsetExists('security')) {
             $record['extra']['user'] = $this->getUsername($this->app['security']);
         }

Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,11 @@ public function __construct(Application $app, $token = null)`
`32`	`32`
`33`	`33`	`public function __invoke(array $record)`
`34`	`34`	`{`
`35`		`- $record['extra']['clientIp'] = $this->getClientIp($this->app['request_stack']);`
	`35`	`+ try {`
	`36`	`+ $record['extra']['clientIp'] = $this->getClientIp($this->app['request_stack']);`
	`37`	`+ } catch (\Exception $e) {`
	`38`	`+ $record['extra']['clientIp'] = '';`
	`39`	`+ }`
`36`	`40`	`if ($this->app->offsetExists('security')) {`
`37`	`41`	`$record['extra']['user'] = $this->getUsername($this->app['security']);`
`38`	`42`	`}`