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

feat: allow publishing with specified IP #2

Closed
wants to merge 1 commit into from
Closed

feat: allow publishing with specified IP #2

wants to merge 1 commit into from

Conversation

keithamus
Copy link

This takes watson#28 and applies it here.

My hope is that I can configure https://github.com/KhaosT/HAP-NodeJS to specify a custom IP as I want to use a proxy in front of my HAP install.

@donavanbecker donavanbecker added the enhancement New feature or request label Apr 9, 2020
@Supereg Supereg self-assigned this Apr 25, 2020
@Supereg
Copy link
Member

Supereg commented Apr 28, 2020

Actually this PR does not work here. self is not defined and the function signatures for rrA and rrAAAA changed to not expect the self argument anymore.
But working on this features anyway.

records.push(rrA(self, this.ip))
records.push(rrAAAA(self, this.ip))

@Supereg Supereg closed this May 11, 2020
@keithamus keithamus deleted the patch-1 branch May 12, 2020 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants