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(doctor): Report connection type changes in rolling fashion #2251

Merged
merged 6 commits into from
May 13, 2024

Conversation

flub
Copy link
Contributor

@flub flub commented Apr 29, 2024

Description

The doctor already shows the current connection type, this changes to show changes in a rolling fashion.

It also unambiguously logs each connection type change now.

This now prints something like:
Connection with gzjlsppiplwpp7pn changed: direct (after 1.548µs)

on each connection change.

Breaking Changes

Notes & open questions

Despite using the conn_type_stream this stream events are still lost, especially very early on in the connection lifetime.

Change checklist

  • Self-review.
  • Documentation updates if relevant.
  • Tests if relevant.
  • All breaking changes documented.

The doctor already shows the current connection type, this changes to
show changes in a rolling fashion.

This is not exact and some switches might be missed since the doctor
mechanism this hooks into is poll based.  A better version would be
using the new conn_type_stream and print each change.
@flub flub marked this pull request as ready for review May 10, 2024 15:32
@flub flub requested a review from dignifiedquire May 10, 2024 15:32
@flub flub enabled auto-merge May 13, 2024 11:48
@flub flub added this pull request to the merge queue May 13, 2024
Merged via the queue into main with commit 9a050a9 May 13, 2024
22 checks passed
@flub flub deleted the flub/doctor-report-conn-type branch May 13, 2024 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants