Skip to content

Conversation

@NataKiss
Copy link
Contributor

Added HaGeZi DNS to Known DNS Providers

Added HaGeZi DNS to Known DNS Providers
Copy link
Contributor

@windsurf-bot windsurf-bot bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 To request another review, post a new comment with "/windsurf-review".

Comment on lines 1259 to 1261
| DNS-over-HTTPS | `https://wurzn.hagezi.org/dns-query` | [Add to AdGuard](adguard:add_dns_server?address=https://wurzn.hagezi.org/dns-query&name=root.hagezi.org), [Add to AdGuard VPN](adguardvpn:add_dns_server?address=<https://wurzn.hagezi.org/dns-query&name=root.hagezi.org)> |
| DNS-over-TLS | `tls://wurzn.hagezi.org` | [Add to AdGuard](adguard:add_dns_server?address=tls://wurzn.hagezi.org&name=root.hagezi.org), [Add to AdGuard VPN](adguardvpn:add_dns_server?address=tls://wurzn.hagezi.org&name=root.hagezi.org) |
| DNS-over-QUIC | `quic://wurzn.hagezi.org` | [Add to AdGuard](adguard:add_dns_server?address=quic://wurzn.hagezi.org&name=root.hagezi.org), [Add to AdGuard VPN](adguardvpn:add_dns_server?address=quic://wurzn.hagezi.org&name=root.hagezi.org) |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'Add to AdGuard' links for the Nuremberg server incorrectly use 'root.hagezi.org' in the name parameter instead of 'wurzn.hagezi.org'. This should be updated for consistency with the server address.

Comment on lines 1267 to 1269
| DNS-over-HTTPS | `https://juuri.hagezi.org/dns-query` | [Add to AdGuard](adguard:add_dns_server?address=<https://juuri.hagezi.org/dns-query&name=root.hagezi.org)>, [Add to AdGuard VPN](adguardvpn:add_dns_server?address=<https://juuri.hagezi.org/dns-query&name=root.hagezi.org)> |
| DNS-over-TLS | `tls://juuri.hagezi.org` | [Add to AdGuard](adguard:add_dns_server?address=tls://juuri.hagezi.org&name=root.hagezi.org), [Add to AdGuard VPN](adguardvpn:add_dns_server?address=tls://juuri.hagezi.org&name=root.hagezi.org) |
| DNS-over-QUIC | `quic://juuri.hagezi.org` | [Add to AdGuard](adguard:add_dns_server?address=quic://juuri.hagezi.org&name=root.hagezi.org), [Add to AdGuard VPN](adguardvpn:add_dns_server?address=quic://juuri.hagezi.org&name=root.hagezi.org) |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The 'Add to AdGuard' links for the Helsinki server incorrectly use 'root.hagezi.org' in the name parameter instead of 'juuri.hagezi.org'. This should be updated for consistency with the server address.


| Protocol | Address | |
|----------------|----------------------------------------------------|----------------|
| DNS-over-HTTPS | `https://root.hagezi.org/dns-query` | [Add to AdGuard](adguard:add_dns_server?address=<https://root.hagezi.org/dns-query&name=root.hagezi.org)>, [Add to AdGuard VPN](adguardvpn:add_dns_server?address=<https://root.hagezi.org/dns-query&name=root.hagezi.org)> |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are unnecessary angle brackets (< and >) in some of the DNS-over-HTTPS URLs (lines 1251, 1259, 1267) which will cause formatting issues in the rendered markdown. These should be removed.

@github-actions
Copy link

Preview was deployed to: https://pull-request-382.kb-dns.pages.dev/

@github-actions
Copy link

Preview was deployed to: https://pull-request-382.kb-dns.pages.dev/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants