php-http / message

HTTP Message related tools
http://php-http.org
MIT License
1.3k stars 42 forks source link

Omit binary body in FullHttpMessageFormatter #128

Closed ostrolucky closed 4 years ago

ostrolucky commented 4 years ago
Q A
Bug fix? no
New feature? no
BC breaks? yes
Deprecations? no
Related tickets fixes #126
Documentation -
License MIT

What's in this PR?

Makes handling for binary body similar to what CurlCommandFormatter does. I was initially playing with idea to use bin2hex, but then I realized people who actually want binary bodies logged will be minority, for majority this is something they are logging unintended by oversight.

Checklist