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.

Sign up for GitHub

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

Jump to bottom

[WIP] feat: add DoQ Method in dns resolver #1011

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

[WIP] feat: add DoQ Method in dns resolver #1011

wants to merge 1 commit into from

Conversation

HynoR
Copy link
Contributor

@HynoR HynoR commented Mar 27, 2024 •
edited
Loading

Now Hysteria can use DoQ Server in resolver. But the code still need to be improved and tested.
About DoQ: https://adguard-dns.io/zh_cn/blog/dns-over-quic.html

config example:

resolver:
  type: quic
  quic:
    addr: dns-unfiltered.adguard.com:853  # or  000000.dns.nextdns.io
    # sni: dns-unfiltered.adguard.com
    # If there are no special requirements,  leave it blank and the hostname in Addr will be used automatically.
    insecure: false

result:
image

既然都有quic-go了，那么干脆也把DoQ DoH3带上吧

TODO:

  • Quic Conntection Management
  • DoQ stability Test
  • DoH3 Support

@haruue haruue added the enhancement New feature or request label Mar 27, 2024
@HynoR HynoR force-pushed the beta/DoQ branch 2 times, most recently from dba455e to 85844e7 Compare March 27, 2024 06:19
@tobyxdd tobyxdd self-requested a review March 27, 2024 17:56
@tobyxdd
Copy link
Collaborator

tobyxdd commented Mar 28, 2024

感谢。这个是还没完成吗？感觉需要处理连接断开重连的情况，不能就一个连接永远用下去。。

@HynoR
Copy link
Contributor Author

HynoR commented Mar 29, 2024 •
edited
Loading

感谢。这个是还没完成吗？感觉需要处理连接断开重连的情况，不能就一个连接永远用下去。。

work in progress啦，doq我自己写完测了还行但是运行久了还是有报错和所以先把草稿和想法发出来看看意见。
如果没问题的话，我是准备doq和doh3都完成了才决定合并。

@HynoR HynoR marked this pull request as draft March 29, 2024 00:12
@HynoR
Copy link
Contributor Author

HynoR commented Mar 29, 2024

现在改成draft了，我去研究一下同行的doq方向的代码

n-WN
n-WN reviewed Jul 15, 2024
View reviewed changes
Copy link

@n-WN n-WN left a comment

Choose a reason for hiding this comment

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

After running for a long time, why does it report an error?

@haruue haruue linked an issue Jul 17, 2024 that may be closed by this pull request
Add H3 (HTTP/3) resolver to Hysteria 2 #1159
Open
@empireofpersia
Copy link

Hi !
When will DoQ & DoH3 be operative ?

@HynoR
Copy link
Contributor Author

HynoR commented Jul 25, 2024

I'll be back when I'm finished with my formal work.

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

@n-WN n-WN n-WN left review comments

@tobyxdd tobyxdd Awaiting requested review from tobyxdd

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add H3 (HTTP/3) resolver to Hysteria 2
5 participants
@HynoR @tobyxdd @empireofpersia @n-WN @haruue