[PR #148] [MERGED] Implement UDP stream timeout. #281

New issue

Closed

opened 2026-02-26 12:34:29 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 12:34:29 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/cbeuw/Cloak/pull/148
Author: @notsure2
Created: 12/22/2020
Status: ✅ Merged
Merged: 12/23/2020
Merged by: @cbeuw

Base: master ← Head: udp-session-timeout

📝 Commits (3)

874d30c Implement UDP stream timeout.
a26a2e8 Use a mutex instead of sync.Map in RouteUDP.
e3ba82c Code style changes and unlock mutex when failed to open stream

📊 Changes

1 file changed (+21 additions, -5 deletions)

View changed files

📝 internal/client/piper.go (+21 -5)

📄 Description

Fixes #147 and also the same case for multiplex, and also a memory leak in case a session sends a packet once and then times out (the old stream would be stuck in the map forever).

@cbeuw Please review and merge thanks 👍

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/cbeuw/Cloak/pull/148 **Author:** [@notsure2](https://github.com/notsure2) **Created:** 12/22/2020 **Status:** ✅ Merged **Merged:** 12/23/2020 **Merged by:** [@cbeuw](https://github.com/cbeuw) **Base:** `master` ← **Head:** `udp-session-timeout` --- ### 📝 Commits (3) - [`874d30c`](https://github.com/cbeuw/Cloak/commit/874d30c5124e69312d09ca144bae699ef0cc71f7) Implement UDP stream timeout. - [`a26a2e8`](https://github.com/cbeuw/Cloak/commit/a26a2e84a4dfbd5e9122198ec3902de3cc831d32) Use a mutex instead of sync.Map in RouteUDP. - [`e3ba82c`](https://github.com/cbeuw/Cloak/commit/e3ba82caac1e1c3f3c9768404b819819cb541d97) Code style changes and unlock mutex when failed to open stream ### 📊 Changes **1 file changed** (+21 additions, -5 deletions) <details> <summary>View changed files</summary> 📝 `internal/client/piper.go` (+21 -5) </details> ### 📄 Description Fixes #147 and also the same case for multiplex, and also a memory leak in case a session sends a packet once and then times out (the old stream would be stuck in the map forever). @cbeuw Please review and merge thanks 👍 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>