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

Specifying port numbers? #4

Open
neonpolaris opened this issue Sep 28, 2015 · 3 comments · May be fixed by #6
Open

Specifying port numbers? #4

neonpolaris opened this issue Sep 28, 2015 · 3 comments · May be fixed by #6

Comments

@neonpolaris
Copy link

The readme encourages specifying fixed port numbers, but gives no indication of how to do that.

This section:
for one_faux in FAUXMOS:
if len(one_faux) == 2:
# a fixed port wasn't specified, use a dynamic one
one_faux.append(0)
seems to suggest adding a field like this:
['nameHere', rest_api_handler('http://addressHere', 'http://addressHere'), 'portHere'],
but this just throws errors and doesn't run.

Can someone help me out here (and perhaps update the readme for others)?

@DoWhileGeek
Copy link

@neonpolaris I'm working on a PR that will replace the lists that define devices with dictionaries that should make things much more concise in code.

I.E.
{"name": "herp derp", "handler": rest_api_handler(...), "port": 1234}

@DoWhileGeek DoWhileGeek linked a pull request Dec 2, 2015 that will close this issue
@charlesshoults
Copy link

When I tried adding the port number like the first mention, the last part of the error I got was "TypeError: an integer is required"

I removed the quotes and it worked fine.

@DoWhileGeek
Copy link

Yea, I'd rather be lazy and complain, than trying to massage values and not always succeeding.

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 a pull request may close this issue.

3 participants