Skip to content

fix: impl<T: Discovery> Discovery for Arc<T>#3495

Merged
ramfox merged 3 commits intomainfrom
ramfox/fix-arc-discovery
Oct 3, 2025
Merged

fix: impl<T: Discovery> Discovery for Arc<T>#3495
ramfox merged 3 commits intomainfrom
ramfox/fix-arc-discovery

Conversation

@ramfox
Copy link
Copy Markdown
Member

@ramfox ramfox commented Oct 3, 2025

Description

Fixes the implementation of Discovery for an Arc<impl Discovery>

  • Added a failing test illustrating that when we added an Arc<Discovery>, the endpoint was unable to actually discover anything
  • Fixed the test by actually implementing the trait on an Arc

closes #3489

Change checklist

  • Self-review.
  • Tests if relevant.

@ramfox ramfox added this to the v0.93.0 milestone Oct 3, 2025
@ramfox ramfox self-assigned this Oct 3, 2025
@ramfox ramfox force-pushed the ramfox/fix-arc-discovery branch from 0a3cabe to 15abcf4 Compare October 3, 2025 02:47
@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 3, 2025

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

Last updated: 2025-10-03T18:47:52Z

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 3, 2025

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: 5e5d9f2

@n0bot n0bot bot added this to iroh Oct 3, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Oct 3, 2025
Co-authored-by: Floris Bruynooghe <flub@n0.computer>
@ramfox ramfox enabled auto-merge October 3, 2025 18:46
@ramfox ramfox added this pull request to the merge queue Oct 3, 2025
Merged via the queue into main with commit f5381bc Oct 3, 2025
29 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Oct 3, 2025
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.

bug: impl Discover for Arc<T> incorrect

2 participants