diff options
| author | Tobias Diaz <tobias.deb@gmail.com> | 2016-08-23 17:13:24 +0200 |
|---|---|---|
| committer | Steve Martinelli <s.martinelli@gmail.com> | 2017-01-11 00:06:15 -0500 |
| commit | b69f3a1f0abcc13f65833d29036028bf3319a2cd (patch) | |
| tree | deb55d642efffa4791c40512d96d5fdf69d85037 /keystoneclient/tests/unit/test_https.py | |
| parent | eb9d39d0f53fac3a4831d7a2b7b688c88d3e512c (diff) | |
| download | python-keystoneclient-b69f3a1f0abcc13f65833d29036028bf3319a2cd.tar.gz | |
Only log application/json content type
This is a combination of 2 commits. The first commit's
message is:
Prevent MemoryError when logging response bodies
Response bodies are loaded into memory prior to
being logged.
Loading huge response bodies may result in a
MemoryError.
This patch proposes that only JSON and TEXT
responses be logged, i.e when the Content-Type
header is application/json or application/text.
Responses that do not include or have a different
Content-Type header will have their body omitted.
This is a sort of backport of the fix for
keystoneauth sessions, see
I93b6fff73368c4f58bdebf8566c4948b50980cee
Co-Authored-By: Samuel de Medeiros Queiroz <samueldmq@gmail.com>
Closes-bug: 1616105
Change-Id: I8f43eee3a0b35041c6cf672e476f8151cf2f8d14
(cherry-picked from: 3e56e0d7e5e1a76d806a3bc1f6d5ef9070f95771)
Only log application/json in session to start
When whitelisting content types to debug print from session we chose
application/json and application/text. application/text is not a real
mime type, text is typically text/plain.
Rather than guess at mime types only print application/json to start
with, but make it easy for additional types to be added later.
Adapted from keystoneauth: Ica5fee076cdab8b1d5167161d28af7313fad9477
Related-Bug: 1616105
Change-Id: Ieaa8fb3ea8d25e09b89498f23b70b18c0f6153f1
(cherry-picked from: 51d16fa344829aadf454faf5e0c4535a8f96a7c8)
Diffstat (limited to 'keystoneclient/tests/unit/test_https.py')
0 files changed, 0 insertions, 0 deletions
