Merge pull request #107 from php-http/captured_body_length

Allow 0 for captured_body_length

Author: dbu

DependencyInjection/Configuration.php

@@ -105,9 +105,8 @@ public function getConfigTreeBuilder()
                             ->defaultValue($this->debug)
                         ->end()
                         ->scalarNode('formatter')->defaultNull()->end()
-                        ->scalarNode('captured_body_length')
+                        ->integerNode('captured_body_length')
                             ->defaultValue(0)
-                            ->canNotBeEmpty()
                             ->info('Limit long HTTP message bodies to x characters. If set to 0 we do not read the message body. Only available with the default formatter (FullHttpMessageFormatter).')
                         ->end()
                     ->end()

Tests/Resources/Fixtures/config/full.php

@@ -16,6 +16,7 @@
     'toolbar' => [
         'enabled' => true,
         'formatter' => 'my_toolbar_formatter',
+        'captured_body_length' => 0,
     ],
     'plugins' => [
         'authentication' => [

Tests/Resources/Fixtures/config/full.xml

@@ -14,7 +14,7 @@
             <uri-factory>Http\Message\UriFactory\GuzzleUriFactory</uri-factory>
             <stream-factory>Http\Message\StreamFactory\GuzzleStreamFactory</stream-factory>
         </classes>
-        <toolbar enabled="true" formatter="my_toolbar_formatter"/>
+        <toolbar enabled="true" formatter="my_toolbar_formatter" captured_body_length="0"/>
         <plugins>
             <authentication>
                 <my_basic type="basic" username="foo" password="bar"/>

Tests/Resources/Fixtures/config/full.yml

@@ -12,6 +12,7 @@ httplug:
     toolbar:
         enabled: true
         formatter: my_toolbar_formatter
+        captured_body_length: 0
     plugins:
         authentication:
             my_basic: