FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

encode strings to utf-8 in ServiceInfo by jgrill · Pull Request #10 · python-zeroconf/python-zeroconf · GitHub

Repository navigation

encode strings to utf-8 in ServiceInfo - #10

Closed
jgrill wants to merge 1 commit into
python-zeroconf:masterfrom
jgrill:master
Closed

jgrill wants to merge 1 commit into
python-zeroconf:masterfrom
jgrill:master

Conversation

jgrill commented Dec 1, 2014

Copy link
Copy Markdown

I found that properties dict values registered as "False" rather than the intended string value, such as "path" in the provided examples.

eg.

Service Paul's Test Web Site._http._tcp.local. added
Type is _http._tcp.local.
Address is 10.0.1.2:80
Weight is 0, Priority is 0
Server is ash-2.local.
Properties are
path: False

I found that properties dict values registered as "False" rather than the intended string value, such as "path" in the provided examples.

eg.

Service Paul's Test Web Site._http._tcp.local. added
  Type is _http._tcp.local.
  Address is 10.0.1.2:80
  Weight is 0, Priority is 0
  Server is ash-2.local.
  Properties are
    path: False

Copy link
Copy Markdown

Coverage decreased (-0.2%) when pulling c2f05f1 on jgrill:master into 05c3c02 on jstasiak:master.

Copy link
Copy Markdown
Collaborator

Thanks for the contribution. I fixed it in f3fd4cd while refactoring the code. Feel free to reopen if it doesn't work for you for any reason.

jstasiak closed this Dec 12, 2014
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

4 participants


Back | FazBrowse Home | New Git URL