Does not work with IPv6?

[Olen]

systemd[1]: Starting systemd-resolved-docker.service - systemd-resolved and docker DNS integration...
systemd-resolved-docker[4018773]: Found gateway 172.17.0.1 for docker0
systemd-resolved-docker[4018773]: Found gateway fd00:ffff::1 for docker0 
systemd-resolved-docker[4018773]: Default domain: docker, allowed domains: .docker 
systemd-resolved-docker[4018773]: Traceback (most recent call last): 
systemd-resolved-docker[4018773]:   File "/usr/bin/systemd-resolved-docker", line 33, in <module>
systemd-resolved-docker[4018773]:     sys.exit(load_entry_point('systemd-resolved-docker==1.0.0', 'console_scripts', 'systemd-resolved-docker')())
systemd-resolved-docker[4018773]:   File "/usr/lib/python3.10/site-packages/systemd_resolved_docker/cli.py", line 60, in main
systemd-resolved-docker[4018773]:     docker_listen_addresses = parse_listen_address(docker_listen_address,
systemd-resolved-docker[4018773]:   File "/usr/lib/python3.10/site-packages/systemd_resolved_docker/utils.py", line 28, in parse_listen_address
systemd-resolved-docker[4018773]:     return default_value()   
systemd-resolved-docker[4018773]:   File "/usr/lib/python3.10/site-packages/systemd_resolved_docker/cli.py", line 61, in <lambda>
systemd-resolved-docker[4018773]:     lambda: [parse_ip_port(entry['gateway']) for entry in
systemd-resolved-docker[4018773]:   File "/usr/lib/python3.10/site-packages/systemd_resolved_docker/cli.py", line 61, in <listcomp>
systemd-resolved-docker[4018773]:     lambda: [parse_ip_port(entry['gateway']) for entry in
systemd-resolved-docker[4018773]:   File "/usr/lib/python3.10/site-packages/systemd_resolved_docker/utils.py", line 21, in parse_ip_port
systemd-resolved-docker[4018773]:     return IpAndPort(ip=ipaddress.ip_address(result.hostname), port=result.port or default_port)
systemd-resolved-docker[4018773]:   File "/usr/lib64/python3.10/ipaddress.py", line 54, in ip_address
systemd-resolved-docker[4018773]:     raise ValueError(f'{address!r} does not appear to be an IPv4 or IPv6 address')
systemd-resolved-docker[4018773]: ValueError: 'fd00' does not appear to be an IPv4 or IPv6 address

I had a quick look to try to find where the rest of the IPv6-address is stripped after it "found gateway", but did not see anything obvious.

[Olen]

There is an issue with the use of urllib.parse:

>>> address="fd00:1:2:3"
>>> result = urllib.parse.urlsplit('//' + address)
>>> result.hostname
'fd00'

>>> address="[fd00:1:2:3]"
>>> result = urllib.parse.urlsplit('//' + address)
>>> result.hostname
'fd00:1:2:3'

Not really sure why urllib.parse is used here at all. Is there any chance the IP-address it finds as a gateway is an url or contains a port number?

[flaktack]

I've added functional IPv6 support in #6. @Olen, could you please take a look and see if it works for you?

DEB/RPM packages are in the artifacts here: https://github.com/flaktack/systemd-resolved-docker/actions/runs/3669375041

Not really sure why urllib.parse is used here at all. Is there any chance the IP-address it finds as a gateway is an url or contains a port number?

No, I've fixed it expect only an IP address as the gateway.

[Olen]

The rpm requires python3.11.
Is the srpm available, so I can rebuild it locally?

[flaktack]

Does the SRPM artifact attached to this action work?
https://github.com/flaktack/systemd-resolved-docker/actions/runs/3670318687

[Olen]

I was able to rebuild it, but I think it was the old version...
The changes from the PR is not in the sources I get from that srpm

[Olen]

I rebuilt it using the source here instead.

I think the issue comes from here:

systemd-resolved-docker/src/systemd_resolved_docker/utils.py

Line 26 in 43e9877

return [parse_ip_port(item) for item in listen_addresses.split(",")]

Which again is called from here:

systemd-resolved-docker/src/systemd_resolved_docker/cli.py

Line 60 in 43e9877

docker_listen_addresses = parse_listen_address(docker_listen_address,

[flaktack]

The SRPM contained to wrong code, could you test the artifact here?
https://github.com/flaktack/systemd-resolved-docker/actions/runs/3670518609

[Olen]

The problem is that parse_ip_port is called from parse_listen_address.

systemd-resolved-docker/src/systemd_resolved_docker/utils.py

Lines 28 to 30 in 353cc03

	def parse_listen_address(listen_addresses, default_value) -> List[IpAndPort]:
	if listen_addresses is not None and len(listen_addresses) > 1:
	return [parse_ip_port(item) for item in listen_addresses.split(",")]

And parse_listen_address is used both for the systemd_resolved_listen_address (which contains a port):

systemd-resolved-docker/src/systemd_resolved_docker/cli.py

Lines 57 to 58 in 353cc03

	systemd_resolved_listen_addresses = parse_listen_address(systemd_resolved_listen_address,
	lambda: [parse_ip_port("127.0.0.153:53")])

And for the docker_listen_addresses, in which docker_gateway do not contain a port:

systemd-resolved-docker/src/systemd_resolved_docker/cli.py

Lines 59 to 61 in 353cc03

	docker_listen_addresses = parse_listen_address(docker_listen_address,
	lambda: [parse_ip(entry['gateway']) for entry in
	docker_gateway])

[flaktack]

Thanks for the links, I'll see what I can do 🙇