-
Notifications
You must be signed in to change notification settings - Fork 390
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
fix: convert []trace.Argument
into map<string,EventData>
#3510
fix: convert []trace.Argument
into map<string,EventData>
#3510
Conversation
a599f8f
to
6e001ab
Compare
be28621
to
8ac236c
Compare
[]trace.Argument
into map<string,EventData>
err = grpcStream.Send(&pb.StreamEventsResponse{Event: eventProto}) | ||
if err != nil { | ||
return err | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ditto
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so, if we can't send is because the connection on the other side isn't there anymore for a a given reason, so we should stop sending and unsubscribe the client. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, sounds reasonable
8ac236c
to
5b5521e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This PRs converts
[]trace.Argument
intomap<string,EventData>
, this code is temporary because later the event structure will be integrated into tracee, but I've created a test for all types to make sure we have cover all supported arguments.