Switch destination hasura url config to 127.0.0.1 to avoid ipv6

[thomas-gerber]

Description

It appears that Node 18, which we recently moved to, favors IPv6 resolution. Given that Airbyte spawns sources and destinations as containers that run outside the Faros CE compose, this causes issues when the destination tries to connect to hasura on ::1:8080 instead of 127.0.0.1:8080. Not sure exactly why - more docker networking expertise needed.

So the current fix is just to switch from localhost to an explicit IPv4 127.0.0.1 in the destination settings, which works.

I am not touching other localhosts in the environment variables, as this behavior appears to be specific to code running on Node in a container. I also check that the mock-data scripts are unaffected (they do not run in a container).

Type of change

(Delete what does not apply)

• Bug fix (non-breaking change which fixes an issue)

Checklist

(Delete what does not apply)

• Have you checked that there aren't other open Pull Requests for the same update/change?
• Have you added an explanation of what your changes do and why you'd like us to include them?

[thomas-gerber]

Note that I am not a big fan of that fix, it just looks painful to turn ipv6 on for docker (seems off by default currently)

[ogusak]

@thomas-gerber - this looks more of an issue of the host OS rather than Node's preference. The new version of Node just does not sort the list of entries returned for localhost name. That is, the first entry is picked from the list of IPs returned by the host OS in whatever order it prefers them to be.

[thomas-gerber]

@thomas-gerber - this looks more of an issue of the host OS rather than Node's preference. The new version of Node just does not sort the list of entries returned for localhost name. That is, the first entry is picked from the list of IPs returned by the host OS in whatever order it prefers them to be.

Not sure: here is my /etc/hosts

thomasgerber@Thomass-MacBook-Pro-3 faros-community-edition % cat /etc/hosts                                     
##
# Host Database
#
# localhost is used to configure the loopback interface
# when the system is booting.  Do not change this entry.
##
127.0.0.1	localhost
255.255.255.255	broadcasthost
::1             localhost
# Added by Docker Desktop
# To allow the same kube context to work on the host and the container:
127.0.0.1 kubernetes.docker.internal
# End of section

It should have resolved it to 127.0.0.1.

[ogusak]

The order is determined by the code that returns that list. And we'd need to test it from the docker container where node runs. ping localhost from that container should show the IP.
nodejs/node#40537 (comment)
https://stackoverflow.com/a/74216410