Skip to content

Conversation

@achingbrain
Copy link
Member

@achingbrain achingbrain commented Nov 4, 2023

Some transports like WebRTC require the target peer id to be present on the multiaddr, so append the peer id before filtering by supported transport.

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if necessary (this includes comments as well)
  • I have added tests that prove my fix is effective or that my feature works

Some transports like webrtc require the target peer id to be present
on the multiaddr, so append the peer id before filtering by
supported transport.
@achingbrain achingbrain requested a review from a team as a code owner November 4, 2023 12:36
@achingbrain achingbrain merged commit bafccd6 into master Nov 6, 2023
@achingbrain achingbrain deleted the fix/prepend-peer-id-before-filtering branch November 6, 2023 08:22
This was referenced Jan 18, 2024
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