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

feat(qlog): extend From<EventType> for EventImportance #1684

Merged
merged 2 commits into from
Dec 13, 2023

Conversation

mxinden
Copy link
Contributor

@mxinden mxinden commented Dec 10, 2023

Extend impl From<EventType> for EventImportance for all TransportEventType enum variants.

Extend `impl From<EventType> for EventImportance` for all `TransportEventType`
`enum` variants.
@mxinden mxinden requested a review from a team as a code owner December 10, 2023 09:38
mxinden added a commit to mxinden/neqo that referenced this pull request Dec 10, 2023
This commit adds support for the qlog [`version_information` QUIC
event](https://quicwg.org/qlog/draft-ietf-quic-qlog-quic-events.html#name-version_information)
on the client.

Depends on mozilla#1504

Depends on cloudflare/quiche#1684

Meta issue: mozilla#528
@LPardue
Copy link
Contributor

LPardue commented Dec 13, 2023

The build failures were due to formatting lint (you can use cargo +fmt nightly locally) I pushed a commit that does that to speed things up this time around.

@LPardue LPardue merged commit 12eebf3 into cloudflare:master Dec 13, 2023
24 checks passed
@LPardue
Copy link
Contributor

LPardue commented Dec 13, 2023

Merged as 12eebf3, thanks!

mxinden added a commit to mxinden/neqo that referenced this pull request Dec 20, 2023
mxinden added a commit to mxinden/neqo that referenced this pull request Dec 27, 2023
This commit adds support for the qlog [`version_information` QUIC
event](https://quicwg.org/qlog/draft-ietf-quic-qlog-quic-events.html#name-version_information)
on the server.

Depends on cloudflare/quiche#1684

Meta issue: mozilla#528
mxinden added a commit to mxinden/neqo that referenced this pull request Dec 27, 2023
This commit adds support for the qlog [`version_information` QUIC
event](https://quicwg.org/qlog/draft-ietf-quic-qlog-quic-events.html#name-version_information)
on the server.

Depends on cloudflare/quiche#1684

Meta issue: mozilla#528
@mxinden
Copy link
Contributor Author

mxinden commented Jan 9, 2024

@LPardue could you release qlog v0.10.1? I would need this pull request and #1682 to proceed with mozilla/neqo#1505 and mozilla/neqo#1531. Neither of these are urgent. Thus in case the release process involves many steps, please ignore this comment.

martinthomson pushed a commit to mozilla/neqo that referenced this pull request Jan 11, 2024
* deps(qlog): update to qlog v0.10.0

This commit upgrades all `neqo-*` crates to use `qlog` `v0.10.0`.

See also `qlog` `v0.10.0` release pull request cloudflare/quiche#1647

* feat(qlog): log version_information on client

This commit adds support for the qlog [`version_information` QUIC
event](https://quicwg.org/qlog/draft-ietf-quic-qlog-quic-events.html#name-version_information)
on the client.

Depends on #1504

Depends on cloudflare/quiche#1684

Meta issue: #528

* Use replace github.com/cloudflare with github.com/mxinden

With cloudflare/quiche#1684 merged, one can use
cloudflare's repo.

* Inline ev_data

* Remove crates.io patch

Patch no longer needed since upgrade to neqo v0.11.0 #1547.
martinthomson pushed a commit to mozilla/neqo that referenced this pull request Jan 16, 2024
* feat(qlog): log version_information on server

This commit adds support for the qlog [`version_information` QUIC
event](https://quicwg.org/qlog/draft-ietf-quic-qlog-quic-events.html#name-version_information)
on the server.

Depends on cloudflare/quiche#1684

Meta issue: #528

* Use WireVersion for client version

* Have create_qlog_trace take &ConnectionIdRef
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants