Skip to content

naive: route HTTP/2 connections through wrapper#4287

Open
H-TTTTT wants to merge 5 commits into
SagerNet:testingfrom
H-TTTTT:fix/naive-http2-wrapper-routing
Open

naive: route HTTP/2 connections through wrapper#4287
H-TTTTT wants to merge 5 commits into
SagerNet:testingfrom
H-TTTTT:fix/naive-http2-wrapper-routing

Conversation

@H-TTTTT

@H-TTTTT H-TTTTT commented Jul 12, 2026

Copy link
Copy Markdown

Summary

  • route Naive HTTP/2 connections through HTTP2ConnWrapper instead of the raw connection
  • verify the router receives the wrapper
  • verify the handler waits for and completes after the route callback

Fixes #4249

Testing

  • go test -v ./protocol/naive -run ^TestNewConnectionRoutesHTTP2Wrapper$ -count=1
  • go test -race ./protocol/naive -run ^TestNewConnectionRoutesHTTP2Wrapper$ -count=1
  • go test ./protocol/naive -run ^TestNewConnectionRoutesHTTP2Wrapper$ -count=100
  • gofmt -d protocol/naive/inbound.go protocol/naive/inbound_test.go
  • git diff --check

@H-TTTTT H-TTTTT force-pushed the fix/naive-http2-wrapper-routing branch from 3d46c59 to a4ea20b Compare July 13, 2026 02:32
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.

Naive inbound: raw connection passed to router bypasses HTTP2Wrapper mutex

2 participants