Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Two fixes #2

Open
wants to merge 37 commits into
base: master
Choose a base branch
from
Open

Two fixes #2

wants to merge 37 commits into from

Conversation

hughescr
Copy link

I fixed two things:

  1. Hostname resolution seems to have been broken -- in the sample Browser.py the call self.r.getServiceInfo(type, name) would always return None, basically because the hostname wasn't being looked up correctly so the thing was always timing out and not filling in service info. I think I fixed this with 7674853 but please check my math on that... It works for me now, but I might be doing something dumb
  2. DNSAddress's repr function was ugly. I made it more prettier.

Mike C. Fletcher and others added 30 commits November 10, 2010 16:10
…query

Command-line tool to test whether multicast messages are working on an interface.
Setup script and MANIFEST.in to generate sdist packages (which can be easy_installed).
…nality working. Also allows replying with server names on queries for A/AAAA names.
… server name for ANY queries, requires an iteration over the set of services for every query, which could be optimized away with a pre-calculated mapping if performance suffers.
Reverting change to make a string being logged have a trailing _ note, upstream changes introduce a change to the external api.
Conflicts:
	zeroconf/dns.py
…name probing working correctly/closer to spec.
…ralize ignored record types, ignore 0 query types (seen on real networks)
…ient requests the same data many times very rapidly (i.e. at greater than .2s frequency); basically just rate-limits the outgoing data for equal response packets
marcosdiez pushed a commit to marcosdiez/pyzeroconf that referenced this pull request May 5, 2018
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