8p / EightPointsGuzzleBundle

⛽️ Integrates Guzzle 6.x, a PHP HTTP Client, into Symfony
MIT License
440 stars 71 forks source link

Update Logger.php #57

Closed mathielen closed 8 years ago

mathielen commented 8 years ago
Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
License MIT

$context['request'] could be null (i.e. when the requested URL could not be resolved - DNS-wise) Which results in:

Type error: Argument 1 passed to EightPoints\Bundle\GuzzleBundle\Log\LogResponse::__construct() must be an instance of Psr\Http\Message\ResponseInterface, null given

florianpreusner commented 8 years ago

Nice! Thanks for the fix! Going to merge it after Travis CI tests and create a new bugfix version (5.0.1).