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

QUIC sniffer: handle multiple initial packets #3802

Merged
merged 2 commits into from
Sep 13, 2024
Merged

QUIC sniffer: handle multiple initial packets #3802

merged 2 commits into from
Sep 13, 2024

Conversation

Fangliding
Copy link
Member

@Fangliding Fangliding commented Sep 12, 2024

#3363
#3363 (comment) 把操作搬了过来 就是一些cv工作加上少许修改 本地暂时还没法确定是否有效

@yuhan6665 yuhan6665 changed the title 尝试修复QUIC Sniff QUIC sniffer: handle multiple initial packets Sep 13, 2024
@yuhan6665
Copy link
Member

感谢 能否在 commit 加上原作者名字?

@Fangliding
Copy link
Member Author

感谢 能否在 commit 加上原作者名字?

done


// copied from github.com/quic-go/quic-go/internal/qtls/cipher_suite_go121.go

type cipherSuiteTLS13 struct {
Copy link
Member

Choose a reason for hiding this comment

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

刚才没注意 这个是不是可以用已有的不用新加?

Copy link
Member Author

Choose a reason for hiding this comment

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

还真是 我找了一圈没有才加的 想着如果重复编译器应该会报错吧 结果大小写不一样

@yuhan6665 yuhan6665 merged commit 781aaee into main Sep 13, 2024
36 checks passed
@yuhan6665
Copy link
Member

Thanks @Fangliding @Vigilans

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