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.

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

transaction trace loggers not trimming all hex data #538

Closed
heifner opened this issue Jun 25, 2022 · 0 comments · Fixed by #549
Closed

transaction trace loggers not trimming all hex data #538

heifner opened this issue Jun 25, 2022 · 0 comments · Fixed by #549
Assignees
Labels
3.1 Candidates 3.1 RC2 OCI OCI working this issue...
Milestone

Comments

@heifner
Copy link
Member

heifner commented Jun 25, 2022

The transaction trace loggers are suppose to trim hex data to limit the amount of logging. This is currently not working correctly.

Example:

debug 2022-06-25T17:38:04.275 nodeos producer_plugin.cpp:649 operator() ] [TRX_TRACE] Speculative execution is ACCEPTING tx: {"id":"cb1fad2b268937b29a4564d15bb8eccc2b7788f0ba774a7fb03b8881e5f2f7b9","block_num":17,"block_time":"2022-06-25T17:38:04.500","producer_block_id":null,"receipt":{"status":"executed","cpu_usage_us":3026,"net_usage_words":878},"elapsed":6068,"net_usage":7024,"scheduled":false,"action_traces":[{"action_ordinal":1,"creator_action_ordinal":0,"closest_unnotified_ancestor_action_ordinal":0,"receipt":{"receiver":"eosio","act_digest":"9a90c525172f87bbac0a6378610727f0fe1d7ebe908df973923d29a1606f9a57","global_sequence":17,"recv_sequence":17,"auth_sequence":[["eosio",17]],"code_sequence":1,"abi_sequence":0},"receiver":"eosio","act":{"account":"eosio","name":"setcode","authorization":[{"actor":"eosio","permission":"active"}],"code_hash":"1f5c063768890aad54a6b7079606e4b683e37d8fcf270dfacd9df05f5569348c","data":{"account":"eosio","vmtype":0,"vmversion":0,"code":{"size":17777,"trimmed_hex":"0061736d01000000019d011a60000060037f7e7f0060027f7e0060027f7f0060057f7e7e7e7e0060047f7e7e7e0060017f017f60017f0060037f7f7f017f6000"}},"hex_data":"0000000000ea30550000f18a010061736d01000000019d011a60000060037f7e7f0060027f7e0060027f7f0060057f7e7e7e7e0060047f7e7e7e0060017f017f60017f0060037f7f7f017f6000017f60027f7f017f60017e0060027e7f0060047e7e7e7e0060027f7f017e6000017e60047e7e7e7e017f60047f7e7e7f0060037f7f7f0060067e7e7e7e7f7f017f60047f7e7f7f0060037e7e7e0060037e7e7f017f60047f7f7e7f0060027e7e0060047f7f7f7f00028e041803656e761469735f666561747572655f616374697661746564000603656e761370726561637469766174655f66656174757265000703656e760c656f73696f5f61737 ...

Looks like a merge issue with #190

@heifner heifner self-assigned this Jun 25, 2022
@heifner heifner added the OCI OCI working this issue... label Jun 25, 2022
heifner added a commit that referenced this issue Jun 25, 2022
…ace logging to launcher logger config to enable full logging during tests.
heifner added a commit that referenced this issue Jun 28, 2022
…-3.1

[3.1] Fix log trimming of hex_data JSON output.
heifner added a commit that referenced this issue Jun 28, 2022
[3.2] Fix log trimming of hex_data JSON output.
ClaytonCalabrese pushed a commit that referenced this issue Jun 30, 2022
…ace logging to launcher logger config to enable full logging during tests.
ClaytonCalabrese pushed a commit that referenced this issue Jun 30, 2022
…ace logging to launcher logger config to enable full logging during tests.
@arhag arhag added this to the Mandel 3.1.0 milestone Jul 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3.1 Candidates 3.1 RC2 OCI OCI working this issue...
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants