Skip to content
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

quiche: enable qlog output #5341

Closed
wants to merge 2 commits into from
Closed

quiche: enable qlog output #5341

wants to merge 2 commits into from

Conversation

@bagder
Copy link
Member

@bagder bagder commented May 5, 2020

quiche has the potential to log qlog files. To enable this, you must
build quiche with the qlog feature enabled cargo build --features qlog. curl then passes a file descriptor to quiche, which takes
ownership of the file. The FD transfer only works on UNIX.

The convention is to enable logging when the QLOGDIR environment is
set. This should be a path to a folder where files are written with the
naming template .qlog.

Co-authored-by: Lucas Pardue
Replaces #5337

@bagder bagder added the HTTP/3 label May 5, 2020
lib/vquic/quiche.c Outdated Show resolved Hide resolved
lib/dynbuf.h Show resolved Hide resolved
@LPardue
LPardue approved these changes May 5, 2020
bagder added 2 commits May 4, 2020
quiche has the potential to log qlog files. To enable this, you must
build quiche with the qlog feature enabled `cargo build --features
qlog`. curl then passes a file descriptor to quiche, which takes
ownership of the file. The FD transfer only works on UNIX.

The convention is to enable logging when the QLOGDIR environment is
set. This should be a path to a folder where files are written with the
naming template <SCID>.qlog.

Co-authored-by: Lucas Pardue
Replaces #5337
@bagder bagder force-pushed the bagder/qlog branch from ee490fa to a7366a8 May 5, 2020
@bagder bagder closed this in 80b9db1 May 5, 2020
@bagder bagder deleted the bagder/qlog branch May 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.