diff options
| author | Ed Tanous <edtanous@google.com> | 2023-05-16 18:57:20 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2023-05-19 18:52:09 +0300 |
| commit | ad2b2eb94a9ec23a6c55f271adfdc989ffc9cb04 (patch) | |
| tree | 10b89713aef2a7e79a39d5f079bc08a5c5105801 /http/http_connection.hpp | |
| parent | eea9c979498cdcd6df68c5fcdf28086e45466672 (diff) | |
| download | bmcweb-ad2b2eb94a9ec23a6c55f271adfdc989ffc9cb04.tar.xz | |
http_connection: Allow empty json objects
Currently http_connection will produce empty body in the response if the
res.jsonValue is empty, including empty array, object.
This makes the output confusing in case a response does contain an empty
object or array.
Change the code to print the json object even if it's empty object or
array.
This patchset was previously reverted because of a regression, but this
regression is fixed in 63529.
Tested on previous commit: With an OEM URL that returns empty array
depending on the system config, the response becomes `[]` instead of
empty.
Signed-off-by: Lei YU <yulei.sh@bytedance.com>
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: I1d1bf01a5277ff1bc953b15d9fc410e10f941e70
Diffstat (limited to 'http/http_connection.hpp')
| -rw-r--r-- | http/http_connection.hpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/http/http_connection.hpp b/http/http_connection.hpp index 24af031e81..213960214f 100644 --- a/http/http_connection.hpp +++ b/http/http_connection.hpp @@ -342,7 +342,7 @@ class Connection : res.setHashAndHandleNotModified(); - if (res.body().empty() && !res.jsonValue.empty()) + if (res.body().empty() && res.jsonValue.is_structured()) { using http_helpers::ContentType; std::array<ContentType, 3> allowed{ |