devgrants icon indicating copy to clipboard operation
devgrants copied to clipboard

Implement MDNS discovery in go per the new spec

Open Stebalien opened this issue 5 years ago • 7 comments

This project is specified here: 011-mdns-go-js.md.

Stebalien avatar Aug 05 '19 18:08 Stebalien

I agree with implementing MDNS discovery according to the new spec should be a priority.

Other than go, the same should happen for JS, also aiming to get the interoperable side of things.

@Stebalien @raulk I think we could include JS in this issue, what do you think? Otherwise, I can create a new issue to track JS.

vasco-santos avatar Aug 07 '19 10:08 vasco-santos

@vasco-santos - I've added the JS label to this issue now.

raulk avatar Aug 07 '19 10:08 raulk

Other than go, the same should happen for JS, also aiming to get the interoperable side of things.

I thought js already implemented the new spec. Is that not the case?

Stebalien avatar Aug 07 '19 17:08 Stebalien

No, a compatibility layer was added a few months ago to get interop functioning with go. Ideally we'd get both implemented together and scrap all of that shenanigans.

jacobheun avatar Aug 07 '19 17:08 jacobheun

Ah, so JS implements one thing, Go implements another, and the spec specifies a third? I guess we finally have a real spec.

Stebalien avatar Aug 07 '19 17:08 Stebalien

Ah, so JS implements one thing, Go implements another, and the spec specifies a third? I guess we finally have a real spec.

Yeah, that's the current state of things

vasco-santos avatar Aug 07 '19 21:08 vasco-santos

Rust implementation implements the spec. Go & JS implementations do not. https://github.com/libp2p/rust-libp2p/issues/1678

semtexzv avatar Jul 27 '20 08:07 semtexzv