tracee icon indicating copy to clipboard operation
tracee copied to clipboard

improve json unmarshaling

Open itaysk opened this issue 4 years ago • 10 comments

partial fix to: #986

itaysk avatar Sep 02 '21 10:09 itaysk

I've fixed those issues, but I should note that pretty-printed arg values (e.g O_WRONLY) will break. I think we need to communicate if it's raw or printed in the arg type

itaysk avatar Sep 26 '21 16:09 itaysk

I've fixed those issues, but I should note that pretty-printed arg values (e.g O_WRONLY) will break. I think we need to communicate if it's raw or printed in the arg type

Here's another option - pass argMetas slice (from processEvents() func) as a pointer argument to prepareArgs() and change the argument type to the correct value when required

yanivagman avatar Sep 26 '21 20:09 yanivagman

Here's another option - pass argMetas slice (from processEvents() func) as a pointer argument to prepareArgs() and change the argument type to the correct value when required

I opened #1029 to track this one

yanivagman avatar Sep 26 '21 20:09 yanivagman

@itaysk what's left for #986 after merging this one?

yanivagman avatar Sep 28 '21 21:09 yanivagman

I don't think we can merge this before #1029 right? I've talked with @grantseltzer about looking into #986 generally

itaysk avatar Sep 29 '21 12:09 itaysk

I don't think we can merge this before #1029 right?

Why not? It resolves current bugs that we have. If #1029 will require changes, it is the responsibility of this future PR to fix it

yanivagman avatar Sep 29 '21 12:09 yanivagman

it creates a regression AFAIU

pretty-printed arg values (e.g O_WRONLY) will break

itaysk avatar Sep 29 '21 13:09 itaysk

@itaysk This PR depends on completing #1029 , right?

yanivagman avatar Dec 01 '21 21:12 yanivagman

@itaysk can you please rebase your changes?

yanivagman avatar Jan 13 '22 12:01 yanivagman

After we have protobuf implemented (and a protobuf wrapper from tracee event to protobuf, so go-cel can work, can be found at https://github.com/aquasecurity/tracee/pull/1766/files#diff-af1a3c488c6c0d521bd322a92cdb79b2eaaf758ee7245db25b5742bb3d8aa32aR95) we will start sorting all the types when marshaling / un-marshaling tracee events since those will be strongly typed.

rafaeldtinoco avatar Jun 24 '22 12:06 rafaeldtinoco

Closing this due to inactivity.

rafaeldtinoco avatar Mar 14 '23 22:03 rafaeldtinoco