Skip to content

feat(iroh): add tracing events for connections#4021

Merged
Frando merged 2 commits intomainfrom
Frando/conn-events
Mar 13, 2026
Merged

feat(iroh): add tracing events for connections#4021
Frando merged 2 commits intomainfrom
Frando/conn-events

Conversation

@Frando
Copy link
Copy Markdown
Member

@Frando Frando commented Mar 13, 2026

Description

This adds iroh::_event::conn events for incoming and outgoing connections. We already have path events, but no connection events, so this makes our event system more complete, and gives good context when reading iroh::_events.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 13, 2026

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/4021/docs/iroh/

Last updated: 2026-03-13T11:16:22Z

@Frando Frando requested a review from flub March 13, 2026 10:28
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 13, 2026

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: 88755d8

Copy link
Copy Markdown
Contributor

@flub flub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yay for more people getting use out of the events!

}
}
fn handle_connection_close(&mut self, conn_id: ConnId) {
fn handle_connection_close(&mut self, conn_id: ConnId, reason: ConnectionError) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could we add the random missing newline in front of this while we're here?

@Frando Frando enabled auto-merge March 13, 2026 11:15
@n0bot n0bot bot added this to iroh Mar 13, 2026
@github-project-automation github-project-automation bot moved this to 🚑 Needs Triage in iroh Mar 13, 2026
@Frando Frando added this pull request to the merge queue Mar 13, 2026
Merged via the queue into main with commit f226881 Mar 13, 2026
31 checks passed
@github-project-automation github-project-automation bot moved this from 🚑 Needs Triage to ✅ Done in iroh Mar 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants