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

Add note regarding current disabled status of mDNS #661

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add note regarding current disabled status of mDNS #661

wants to merge 1 commit into from

Conversation

cco3
Copy link
Contributor

@cco3 cco3 commented May 19, 2016

No description provided.

@cco3
Copy link
Contributor Author

cco3 commented May 19, 2016

#657

@@ -1,5 +1,7 @@
## mDNS Support

**Due to several issues, mDNS support is currently disabled this app! We intend to revisit (and hopefully reenable) this feature in the future.**
Copy link
Contributor

Choose a reason for hiding this comment

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

disabled *in this app

IMO "re-enable" reads better than "reenable"

@raggi
Copy link

raggi commented May 19, 2016

I'm also wondering if this is true of the android chrome implementation. I didn't have an mdns server to hand that would allow me to generate the structure that avahi would, as the one I had was too strict.

Across the board, it'd be nice to know what is and isn't supported by the chrome implementation, especially as the chrome implementation links to this repo in docs. It appears by experimentation that ssdp and bluetooth work in chrome the way they are described here.

@cco3
Copy link
Contributor Author

cco3 commented May 19, 2016

The android chrome implementation doesn't have mdns support. This is a test bed and will typically have always have features not in the chrome implementation. ssdp also doesn't work in chrome...not sure why it appears to for you.

I'm not sure the docs in this repo are the best place to describe the feature sets of other PW clients. It's something we can consider though.

@raggi
Copy link

raggi commented May 20, 2016

e.g. https://support.google.com/chrome/answer/6239299 links to this repo.

I thought I saw something from developers.google.com too, but I may have got my wires crossed.

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.

4 participants