Execution log write functionality #38
Replies: 1 comment
-
@iamkelllly currently I don't think execution logs provide a lot of value because of the unaddressed issues specified above. These old bug still needs to be fixed https://ethyca.atlassian.net/browse/SOL-271. When the logs are being created, we're not using all the information available to us to paint an accurate picture about what fields are affected, the logs are too broad. Additionally, the current state has each action getting its own row, we are not currently overriding the same record for each log. I find this really confusing to read, it's hard to tie together which logs are related to each other, and it generates so many logs. I'd rather see one log for each node that we're constantly updating, but if we want all the history, then a ticket to group related logs together in the display would be handy. |
Beta Was this translation helpful? Give feedback.
-
Context: https://github.com/ethyca/solon/pull/179#discussion_r738854293
We are looking to improve the way that execution logs write to the database. In the testing of the current implementation, we found that:
fields_affected
in the logs incorrectly include data categories that were not specified on the erasure ruleThe purpose of the execution logs is to elucidate:
This differs from application logs, where the purpose is to be view holistic application activity (ie: authentication logs, retry logs, etc)
For implementing the next generation of execution logs, the open questions are:
Beta Was this translation helpful? Give feedback.
All reactions