Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Json tracing updates and fixes (enable in all clusters, fix lists and large payloads) #28022

Merged
merged 17 commits into from
Jul 19, 2023
Merged

Json tracing updates and fixes (enable in all clusters, fix lists and large payloads) #28022

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
ef21b0c
Fix json tracing
andreilitvin Jul 18, 2023
e74caed
Increase the buffer size for json tracing as well
andreilitvin Jul 18, 2023
b44711e
Full json trace decoding for all clusters app on linux
andreilitvin Jul 18, 2023
b27b0a6
Better formatting of lists - make indices make sense
andreilitvin Jul 18, 2023
19a5842
Restyle
andreilitvin Jul 18, 2023
a87c2f6
Updated comments based on code review
andreilitvin Jul 18, 2023
7217570
Restyled by whitespace
restyled-commits Jul 18, 2023
744be46
Add error logging when exit container fails
andreilitvin Jul 18, 2023
049eb6e
Fix typo
andreilitvin Jul 18, 2023
a2c0b61
Update src/tracing/json/json_tracing.cpp
andy31415 Jul 18, 2023
6740d80
rename [] to Anonymous<> for lists
andreilitvin Jul 18, 2023
13b43d0
Merge remote-tracking branch 'origin/fix_json_tracing' into fix_json_…
andreilitvin Jul 18, 2023
38a9bc5
Rename exit to end for containers
andreilitvin Jul 18, 2023
0a36093
Use a `@` format instead of -
andreilitvin Jul 18, 2023
fbfdee4
One more anonymous update
andreilitvin Jul 18, 2023
7e97b61
Fix unit tests
andreilitvin Jul 18, 2023
8d6cdee
Merge branch 'master' into fix_json_tracing
andy31415 Jul 18, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix typo
  • Loading branch information
@andreilitvin
andreilitvin committed Jul 18, 2023
commit 049eb6ea069b6c7e49fcab8b85d955280288efdc
2 changes: 1 addition & 1 deletion src/lib/format/protocol_decoder.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ void PayloadDecoderBase::ExitContainer(PayloadEntry & entry)
if (err != CHIP_NO_ERROR)
{
mValueBuilder.AddFormat("ERROR: %" CHIP_ERROR_FORMAT, err.Format());
mNameBuilder.Format("EXIT CONTAINER");
mNameBuilder.AddFormat("EXIT CONTAINER");
andy31415 marked this conversation as resolved.
Show resolved Hide resolved
entry = PayloadEntry::SimpleValue(mNameBuilder.c_str(), mValueBuilder.c_str());
mState = State::kDone;
return;
Expand Down
Loading