Open agrover opened 4 years ago
@vonasek mentioned that it might be useful if qlog included payload. This could help make using tools like Wireshark less needed.
Since payload is 1) large and 2) possibly sensitive, we probably don't want to do this by default when doing qlog.
Working on this. Currently blocked by https://github.com/cloudflare/quiche/issues/785
Thanks!
FYI: Patch accepted to fix qlog issue. Upstream will roll a new crate asap and then we can continue work on this issue.
@vonasek mentioned that it might be useful if qlog included payload. This could help make using tools like Wireshark less needed.
Since payload is 1) large and 2) possibly sensitive, we probably don't want to do this by default when doing qlog.