qlog icon indicating copy to clipboard operation
qlog copied to clipboard

Add IP addresses to datagram_* events

Open rmarx opened this issue 4 years ago • 2 comments

We currently extracted these to other events, assuming they wouldn't change or only change sparingly at specific times, handled by other events (e.g., when doing migration).

However, this is not always the case. For example, @huitema mentioned:

NAT rebinding, probes of packets before migration

For these cases, it would be interesting to have (optional) IP from/to addresses in the datagram_* events as well.

rmarx avatar Feb 26 '20 09:02 rmarx

If you want to cover migration, you probably want to include the port numbers too

LPardue avatar Jul 25 '22 16:07 LPardue

yeah I discussed this with Robin. He recommended considering a PathInfo struct that basically contains the 4 tuple and it can be reused for all cases, including Connection Migration and Multi-path

lnicco avatar Aug 04 '22 01:08 lnicco

Solved by https://github.com/quicwg/qlog/pull/336

rmarx avatar Mar 19 '24 11:03 rmarx