This is a patch from Julien Sentier on the mailing list that makes ICMP payloads available at the scripting layer. Is there a reason this isn't already available? I would have committed it to fastpath except I don't know if it's not already doing this due to the potential overhead of creating a lot of strings in ICMP floods. At the very least, I suppose it could be optional (which the patch doesn't currently do).
Replying to [seth|comment:11]:
> Did this happen? I didn't track the ICMP changes closely.
No icmp_sent events still don't include a payload parameter.
Bumping this then. It's too big of a change to include now.
I'll just add a new icmp_sent_payload event for this to address the overhead concern.