Make field "Bootstrap DNS servers" accept upstream syntax

[Ziris85]

Prerequisites

• I have checked the Wiki and Discussions and found no answer

• I have searched other issues and found no duplicates

• I want to report a bug and not ask a question or ask for help

• I have set up AdGuard Home correctly and configured clients to use it. (Use the Discussions for help with installing and configuring clients.)

Platform (OS and CPU architecture)

Linux, AMD64 (aka x86_64)

Installation

Snapcraft

Setup

On one machine

AdGuard Home version

0.107.36

Action

I attempted to add comments to the "Bootstrap DNS servers" text box. Attempting to apply or test the upstreams results in an error:

Error: control/dns_config | checking bootstrap # 9.9.9.10: invalid address: bootstrap # 9.9.9.10:53: ParseAddr("# 9.9.9.10"): unexpected character (at "# 9.9.9.10") | 400

Contents that I'm testing with are:

# 9.9.9.10
149.112.112.10
2620:fe::10
2620:fe::fe:10

Expected result

The "Examples" section immediately above the textbox indicate that lines leading with # are proper syntax for comments, so I would expect the syntax to validate and for AdGuard to ignore the lines indicated as comments

Actual result

AdGuard fails to validate the config and refuses to apply the settings

Additional information and/or screenshots

No response

[Ziris85]

Realized my initial config wasn't actually valid, so tested again with valid configs and verified the issue is still present. Updated initial comment with what should otherwise be valid configs

[ainar-g]

The bootstrap field only really accepts IP addresses, if I recall correctly. It's not a proper upstream config currently, so no comments and no domain-specific rules. At the very least, the UI needs to be updated to make that clear.

[ainar-g]

The UI changes will arrive in v0.107.37. I will retitle this issue to be about the support for the full upstream syntax, including comments.