tree 6b3649a76b20cbe212f19997de71e571037bad65
parent ad2b2eb94a9ec23a6c55f271adfdc989ffc9cb04
author Ed Tanous <edtanous@google.com> 1684262358 -0700
committer Ed Tanous <ed@tanous.net> 1684511529 +0000

Clear json object

nlohmann::json::clear() has different behavior dependent on what the
underlying object is, rather than doing the expected behavior of
completely clearing the json object.  This didn't matter because of a
similar bug in http_connection that relied on nlohmann:json::empty()
which is ALSO type dependent, so these worked.

Unfortunately, in 02e01b5108d46720a0b438c0d79952464320d954 we wanted to
allow empty objects, and this bug was exposed.

There are two places where clear() is used, once in Response, which is
clearly not the intent, which is to reset the object to the original
constructed state.  The other place we call clear is in Manager, where
we use it to clear incremental results.  That was a previous best practice that has been eliminated everywhere else (now we return as
many results with the error as we are able).  It has been removed.

Tested: Logging into the webui in firefox no longer core dumps.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ic89a037b30fb40c0e6eaeb939cae6e006dd0ffac