Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Link-local IPv6 resolvers fail #43

Closed
james-callahan opened this issue Mar 1, 2018 · 3 comments · Fixed by #47
Closed

Link-local IPv6 resolvers fail #43

james-callahan opened this issue Mar 1, 2018 · 3 comments · Fixed by #47
Labels

Comments

@james-callahan
Copy link

james-callahan commented Mar 1, 2018

My resolv.conf:

# Generated by NetworkManager
nameserver 192.168.1.254
nameserver 8.8.8.8
nameserver fe80::1%enp0s20f0u1u1
# NOTE: the libc resolver may not support more than 3 nameservers.
# The nameservers listed below may not be recognized.
nameserver fd80::1e5f:2bff:fe92:95c9

Results in:

2018/03/01 15:15:32 [error] 28930#0: *9 [lua] responses.lua:121: after(): failed the initial dns/balancer resolve for 'mockbin.org' with: dns lookup pool exceeded retries (1): failed to create a resolver: failed to set peer name: failed to parse host name "[fe80::1%enp0s20f0u1u1]": invalid IPv6 address, client: 127.0.0.1, server: kong, request: "GET / HTTP/1.1", host: "example.com"

Running version 1.0.0-1

@Tieske
Copy link
Member

Tieske commented Mar 12, 2018

The underlying open resty lib doesn't support zone indices, and neither does this library.

background info: https://en.wikipedia.org/wiki/IPv6_address#Scoped_literal_IPv6_addresses

@james-callahan
Copy link
Author

The underlying open resty lib doesn't support zone indices, and neither does this library.

In that case, nameservers with a ipv6 scope should be skipped (rather than causing dns lookups to fail)

@Tieske
Copy link
Member

Tieske commented Mar 13, 2018

Yes that would for now probably be the best option

Tieske added a commit that referenced this issue May 18, 2018
Provide a nice log message and skip, instead of erroring out

fixes #43
Tieske added a commit that referenced this issue May 18, 2018
Provide a nice log message and skip, instead of erroring out

fixes #43
Tieske added a commit that referenced this issue May 18, 2018
Provide a nice log message and skip, instead of erroring out

fixes #43
Tieske added a commit that referenced this issue May 18, 2018
Provide a nice log message and skip, instead of erroring out

fixes #43
Tieske added a commit that referenced this issue May 18, 2018
Provide a nice log message and skip, instead of erroring out

fixes #43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants