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

IPv6 support #557

Closed
wants to merge 4 commits into from
Closed

IPv6 support #557

wants to merge 4 commits into from

Conversation

petrosagg
Copy link
Contributor

@petrosagg petrosagg commented Feb 21, 2018

In order for the supervisor to work correctly in IPv6 environments it needs to be RFC 3484 compliant. This RFC is implemented in the operating system and when a hostname resolution is done the results are returned to the application in the order of highest likelihood of succeeding.

NodeJS doesn't respect that order and re-sorts the records. Furthermore, it will only try the first entry and will not continue attempting the rest on failure.

To fix this we need to always pass the verbatim: true flag when resolving that was introduced in nodejs 8.5.0 and will become the default in the next major version. See nodejs/node#14731 and nodejs/node#17793

@ghost ghost assigned petrosagg Feb 21, 2018
@ghost ghost added the flow/in-review label Feb 21, 2018
@ghost
Copy link

ghost commented Feb 22, 2018

@petrosagg, status checks have failed for this PR. Please make appropriate changes and recommit.

@ghost
Copy link

ghost commented Feb 23, 2018

@petrosagg, status checks have failed for this PR. Please make appropriate changes and recommit.

Switch from meta-nodejs to OpenEmbedded since it now officially maintained.
Also updated the base system to rocko and bitbabke to 1.36

Signed-off-by: Petros Angelatos <petrosagg@gmail.com>
Signed-off-by: Petros Angelatos <petrosagg@gmail.com>
Signed-off-by: Petros Angelatos <petrosagg@gmail.com>
Signed-off-by: Petros Angelatos <petrosagg@gmail.com>
@petrosagg petrosagg changed the title WIP: IPv6 support IPv6 support Mar 5, 2018
@ghost
Copy link

ghost commented Mar 5, 2018

@petrosagg, status checks have failed for this PR. Please make appropriate changes and recommit.

@pcarranzav
Copy link
Contributor

Superseded by #566

@pcarranzav pcarranzav closed this Mar 7, 2018
@ghost ghost removed the flow/in-review label Mar 7, 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