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

Fatal Error: DNSCrypt DNS-over-QUIC, Anonymized DNS #2674

Closed
Eyeborgs opened this issue Feb 13, 2021 · 8 comments
Closed

Fatal Error: DNSCrypt DNS-over-QUIC, Anonymized DNS #2674

Eyeborgs opened this issue Feb 13, 2021 · 8 comments
Assignees
Milestone

Comments

@Eyeborgs
Copy link

Eyeborgs commented Feb 13, 2021

Попробовал сделать DNSCrypt через DNS-over-QUIC и DNSCrypt Anonymized DNS и после этого AGH полностью убивался - в Windows помогало только вернуть AdGuardHome.yaml{я просто делаю резервную копию} и после переустановить службу AdGuardHome.exe --service uninstall а после AdGuardHome.exe --service install и все тогда начало работать - если просто службу запустить после фатальной ошибки не чего не выходит и служба просто останавливается...

Протокол DNSCrypt через DNS-over-TLS работает нормально

Issue Details

  • Version of AdGuard Home server:
    • v0.105.0
  • Operating system and version:
    • Windows 10, Windows Server

Screenshots

Screenshot:

DNSCrypt через DNS-over-QUIC
DNSCrypt-DNS-over-QUIC
DNSCrypt Anonymized DNS
image
Лог из CMD
image

@ameshkov
Copy link
Member

Ну Anonymized DNSCrypt мы просто не поддерживаем, может в этом дело?

@Eyeborgs
Copy link
Author

@ameshkov Ну а почему она ломает весть AGH ?

@ameshkov
Copy link
Member

А вы через интерфейс задавали или через yaml?

@Eyeborgs
Copy link
Author

@ameshkov через интерфейс

@ameshkov ameshkov added this to the v0.105.2 milestone Feb 15, 2021
@ameshkov
Copy link
Member

@ainar-g @EugeneOne1 не хватает минимальной валидации апстримов - надо хотя бы проверять, что мы можем распарсить нормально апстрим.

adguard pushed a commit that referenced this issue Feb 16, 2021
Merge in DNS/adguard-home from 2674-quic-upstream to master

Updates #2674.

Squashed commit of the following:

commit ce76435
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 18:23:39 2021 +0300

    all: fix log

commit b53e436
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 18:21:59 2021 +0300

    all: log changes

commit 1cdca48
Merge: bc441da e32c18f
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 18:15:47 2021 +0300

    Merge branch 'master' into 2674-quic-upstream

commit bc441da
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 17:06:47 2021 +0300

    dnsforward: fix error wrapping

commit 281c1b4
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 17:00:35 2021 +0300

    all: fix validation, imp shutdown

commit 31a5ea7
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 14:53:10 2021 +0300

    dnsforward: fix upstreams validation
@EugeneOne1
Copy link
Member

Снэпшот 1122e71 должен решить возникшую у вас проблему, но, к сожалению, лишь частично. Дело в том, что одна из наших библиотек все еще не дает возможность настроить DNSCrypt через DNS-over-QUIC. Тем не менее, ожидается, что AdGuard Home больше не будет требовать переустановки, чтобы запуститься после ошибки, которую вы нашли. Не могли бы вы проверить, решилась ли возникшая проблема у вас?

@Eyeborgs
Copy link
Author

@EugeneOne1 проверил, теперь AGH не убивается после ошибки.

@EugeneOne1
Copy link
Member

@Eyeborgs, благодарим за тестирование. Проблему с DNS-over-QUIC мы проинспектируем и постараемся решить в скором времени. Можете открыть новый тикет, если у вас возникнет эта или любая другая проблема.

heyxkhoa pushed a commit to heyxkhoa/AdGuardHome that referenced this issue Mar 20, 2023
Merge in DNS/adguard-home from 2674-quic-upstream to master

Updates AdguardTeam#2674.

Squashed commit of the following:

commit ce76435
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 18:23:39 2021 +0300

    all: fix log

commit b53e436
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 18:21:59 2021 +0300

    all: log changes

commit 1cdca48
Merge: bc441da e32c18f
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 18:15:47 2021 +0300

    Merge branch 'master' into 2674-quic-upstream

commit bc441da
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 17:06:47 2021 +0300

    dnsforward: fix error wrapping

commit 281c1b4
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 17:00:35 2021 +0300

    all: fix validation, imp shutdown

commit 31a5ea7
Author: Eugene Burkov <e.burkov@adguard.com>
Date:   Tue Feb 16 14:53:10 2021 +0300

    dnsforward: fix upstreams validation
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants