mirror of
https://github.com/cbeuw/Cloak.git
synced 2026-04-26 13:05:56 +03:00
[PR #104] [MERGED] Fix: server was timing out response from the proxy instead of the request from the client. #266
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Cloak#266
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/cbeuw/Cloak/pull/104
Author: @notsure2
Created: 4/4/2020
Status: ✅ Merged
Merged: 4/4/2020
Merged by: @cbeuw
Base:
master← Head:premature-tcp-close📝 Commits (1)
08446d6Fix: server was timing out response from the proxy instead of the request from the client.📊 Changes
1 file changed (+2 additions, -2 deletions)
View changed files
📝
cmd/ck-server/ck-server.go(+2 -2)📄 Description
This fixes a very annoying subtle bug, where the timeout was flipped on the client side in case of TCP only. So in case of a server resource that replies with empty TCP ACKs when receiving input (eg: idle SSH session sending SSH null packets to keep the connection alive), cloak was timing out the connection and killing it.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.