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

Install: Check for valid upstream DNS IP addresses excludes IPv6 #5559

Open
voltagex opened this issue Jan 26, 2024 · 4 comments · May be fixed by #5560
Open

Install: Check for valid upstream DNS IP addresses excludes IPv6 #5559

voltagex opened this issue Jan 26, 2024 · 4 comments · May be fixed by #5560
Labels
Bug: Confirmed Verified as a bug to be resolved

Comments

@voltagex
Copy link

https://discourse.pi-hole.net/t/setdns-in-basic-install-sh-does-not-support-ipv6-ipv6-upstream-dns-means-i-cant-finish-the-install/67890

Versions

  • Pi-hole: not installed yet
  • AdminLTE: not installed yet
  • FTL: not installed yet

Platform

  • OS and version: Debian Trixie aarch64
  • Platform: VM

Expected behavior

I am able to install Pi-Hole with PIHOLE_SKIP_OS_CHECK=true and upstream DNS set to IPv6 address/es

A clear and concise description of what you expected to happen.

Actual behavior / bug

image

A clear and concise description of what the bug is.

Steps to reproduce

Steps to reproduce the behavior:

  1. curl -sSL https://install.pi-hole.net | sudo PIHOLE_SKIP_OS_CHECK=true bash
  2. Select custom upstream DNS
  3. Enter 2001:0DB8::123:456 (NOTE: this is an IPv6 documentation address and won't work as a real server!)
  4. See error
@voltagex
Copy link
Author

Note that the UI once Pi-Hole is installed seems to work

image

@pralor-bot
Copy link

This issue has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/setdns-in-basic-install-sh-does-not-support-ipv6-ipv6-upstream-dns-means-i-cant-finish-the-install/67890/1

@yubiuser
Copy link
Member

Thanks for this report.

There are two issues here: one is expected and one is a bug.
Expected is that you need to overwrite the OS check - Trixi is not an supported OS.


The IPv6 issue is a real bug. As you already found out, we need to check also for valid IPv6 addresses at

if ! valid_ip "${PIHOLE_DNS_1}" || [[ ! "${PIHOLE_DNS_1}" ]]; then

@yubiuser yubiuser added the Bug: Confirmed Verified as a bug to be resolved label Jan 26, 2024
@DL6ER DL6ER linked a pull request Jan 26, 2024 that will close this issue
1 task
@DL6ER
Copy link
Member

DL6ER commented Jan 26, 2024

Should be fixed by #5560

@yubiuser yubiuser linked a pull request Jan 28, 2024 that will close this issue
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Confirmed Verified as a bug to be resolved
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants