74 #261

New issue

Closed

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

kerem commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/cbeuw/Cloak/pull/81
Author: @notsure2
Created: 12/14/2019
Status: ✅ Merged
Merged: 12/14/2019
Merged by: @cbeuw

Base: master ← Head: server-add-streams-sync

📝 Commits (1)

448908f Store stream in packet receive path instead of asynchronously to fix https://github.com/cbeuw/Cloak/issues/74

📊 Changes

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

View changed files

📝 internal/multiplex/session.go (+1 -1)

📄 Description

This is a simple change that fixes bug https://github.com/cbeuw/Cloak/issues/74

The cause of the bug was a race condition between storing the created stream in the session and reading the next packet for this stream on the server.

_{🔄 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/81 **Author:** [@notsure2](https://github.com/notsure2) **Created:** 12/14/2019 **Status:** ✅ Merged **Merged:** 12/14/2019 **Merged by:** [@cbeuw](https://github.com/cbeuw) **Base:** `master` ← **Head:** `server-add-streams-sync` --- ### 📝 Commits (1) - [`448908f`](https://github.com/cbeuw/Cloak/commit/448908fa6e2a03d685e278f12e7f27f19a356ca6) Store stream in packet receive path instead of asynchronously to fix https://github.com/cbeuw/Cloak/issues/74 ### 📊 Changes **1 file changed** (+1 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `internal/multiplex/session.go` (+1 -1) </details> ### 📄 Description This is a simple change that fixes bug https://github.com/cbeuw/Cloak/issues/74 The cause of the bug was a race condition between storing the created stream in the session and reading the next packet for this stream on the server. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>