Skip to content
This repository has been archived by the owner on Oct 25, 2022. It is now read-only.

Integrate mDNS passive discovery #145

Closed
wants to merge 19 commits into from
Closed

Integrate mDNS passive discovery #145

wants to merge 19 commits into from

Conversation

polarbeargo
Copy link
Contributor

@polarbeargo polarbeargo commented Jan 26, 2022

Summary of changes
Changes introduced in this pull request:

  • Integrate async-std, futures, tokio, env_logger for test modules.
    For M5 - implement mDNS discovery #134 currently only have
  • feat: mdns passive discovery.
  • feat: impl Event process for Mdns.
  • feat: impl Event process for Gossipsub.
  • feat: impl Event process for Identify.
  • feat: impl Event process for ping.
  • feat: struct NetworkBehaviour for Event process.
  • test: implement tests for mdns passive discovery.
    Reference issue to close (if applicable)

Closes #134
Screenshot (454)

Other information and links

@polarbeargo polarbeargo added this to the m5 milestone Jan 26, 2022
@polarbeargo polarbeargo self-assigned this Jan 26, 2022
@polarbeargo polarbeargo marked this pull request as draft January 26, 2022 08:54
@codecov-commenter
Copy link

codecov-commenter commented Jan 26, 2022

Codecov Report

Merging #145 (aa94401) into main (d78d78f) will decrease coverage by 1.39%.
The diff coverage is 28.57%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #145      +/-   ##
==========================================
- Coverage   85.11%   83.71%   -1.40%     
==========================================
  Files          84       85       +1     
  Lines        3312     3396      +84     
==========================================
+ Hits         2819     2843      +24     
- Misses        493      553      +60     
Impacted Files Coverage Δ
network/src/p2p/event_process.rs 28.57% <28.57%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d78d78f...aa94401. Read the comment docs.

@polarbeargo polarbeargo marked this pull request as ready for review February 8, 2022 12:03
@willemolding
Copy link
Contributor

Branch is getting pretty stale and with networking not on our immediate roadmap I think it will be easier to re-implement when we need it.

@hanabi1224 hanabi1224 deleted the libp2p_mdns branch April 1, 2022 07:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

M5 - implement mDNS discovery
3 participants