app/dispatcher/default.go: Close link when routedDispatch() failed #5131
Conversation
|
把 Mux 等其它入站改成 DispatchLink() 就好了，不需要改这里，不然以后又要删掉多余的代码 |
|
不过 XUDP 那处似乎不好改，那这处还是可以有的，话说 Mux 带了超时机制吗 |
|
No, in server-side-mux and for each session, timeout controlled by the (demuxed)outbound. |
|
话说 #5100 是有必要的吗，似乎等底层 TCP 连接断了就行？写数据出错的话也会自动断吧 |
|
|
|
貌似没有，本来是想让它底层 conn 断了自动 cancel 的，但它返回后才会调用，由于 DispatchLink()，这个权限被传递给最后一级了 |
|
然后 mux 这里由于没传递给最后一级，就需要 mux 本身来处理“出错->返回”这件事 |
|
|
|
总之以上讨论几乎与这个 PR 无关，先合了吧 |
RPRX
changed the title
|
also, ss2022 outbound does not implement but because it uses "sagernet/sing" pkg, it is impossible to implement timer and update the timer after each send/receive-packet(unless we use our own modified-pkg)
|
|
ss2022 was planned to get removed because of license issues #3272 |
Revert "v25.10.15" This reverts commit b69a376. Revert "VLESS Reverse Proxy: Transfer real Source & Local (IP & port), enabled by default" This reverts commit 12f4a01. Revert "XHTTP client: Change default `maxConcurrency` to 1 for speed testing" This reverts commit 9cc7907. Revert "Router: Use built-in-dns only once for all rules (in "IPOnDemand"/"IPIfNonMatch" mode) (XTLS#5210)" This reverts commit 21a9658. Revert "README.md: Add PasarGuard to Web Panels (XTLS#5224)" This reverts commit 7f436f5. Revert "Update github.com/xtls/reality to 20251014195629" This reverts commit dcfde8d. Revert "Bump golang.org/x/net from 0.44.0 to 0.46.0 (XTLS#5215)" This reverts commit 898db92. Revert "Bump google.golang.org/grpc from 1.75.1 to 1.76.0 (XTLS#5212)" This reverts commit 8dd0e38. Revert "transport/internet/reality/reality.go: Safely get negotiated CurveID in VerifyPeerCertificate()" This reverts commit 40f0a54. Revert "Bump github.com/refraction-networking/utls from 1.8.0 to 1.8.1 (XTLS#5229)" This reverts commit 1762d6c. Revert "Fix shadowsocks2022 memory leak (XTLS#5166)" This reverts commit 1952488. Revert "fix: darwin arm64 always has AESGCMHardwareSupport (XTLS#5176)" This reverts commit 4a825c0. Revert "Bump github.com/quic-go/quic-go from 0.54.1 to 0.55.0 (XTLS#5208)" This reverts commit 514c9e5. Revert "feat(config): add unix socket HTTP config loader support (XTLS#5200)" This reverts commit 2f366ae. Revert "Fix vless reverse panic in vision (XTLS#5189)" This reverts commit c0c88f3. Revert "Bump github.com/quic-go/quic-go from 0.54.0 to 0.54.1 (XTLS#5180)" This reverts commit d0344bc. Revert "Bump google.golang.org/protobuf from 1.36.9 to 1.36.10 (XTLS#5203)" This reverts commit a6ebb30. Revert "Outbound: One endpoint and at most one user only (XTLS#5144)" This reverts commit fe57507. Revert "Config: Outbound proxy config no need to be nested (XTLS#5124)" This reverts commit 83c5370. Revert "Bump google.golang.org/grpc from 1.75.0 to 1.75.1 (XTLS#5129)" This reverts commit 1a48453. Revert "app/dispatcher/default.go: Close link when routedDispatch() failed (XTLS#5131)" This reverts commit 3167e5c. Revert "app/dispatcher/default.go: Add comment on run-time rejecting non-existent outbound tag" This reverts commit 5148c57.
routedDispatchis also used for(pipe)Dispatch(it is still used in some cases) so link should close immediately whenroutedDispatchfailed.otherwise, inbound/mux remain open without doing anything.