[PR #82] [MERGED] Make tcp keepalive optional on client -> server and server -> proxy connections, controllable via config. #262

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

📋 Pull Request Information

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

Base: masterHead: tcp-keepalive


📝 Commits (1)

  • 7c315bf Make keepalive optional on client -> server and server -> proxy connections. Use KeepAlive value in config (seconds).

📊 Changes

4 files changed (+22 additions, -6 deletions)

View changed files

📝 cmd/ck-client/ck-client.go (+2 -2)
📝 cmd/ck-server/ck-server.go (+2 -1)
📝 internal/client/state.go (+7 -0)
📝 internal/server/state.go (+11 -3)

📄 Description

Hello,
This pull request just makes the default tcp keepalive since GO 1.13 of 15 seconds explicit.
It is enabled on the client app, in the client cloak --> remote cloak direction
and it is enabled on the server app in the server cloak --> server proxy direction.


🔄 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/82 **Author:** [@notsure2](https://github.com/notsure2) **Created:** 12/14/2019 **Status:** ✅ Merged **Merged:** 4/4/2020 **Merged by:** [@cbeuw](https://github.com/cbeuw) **Base:** `master` ← **Head:** `tcp-keepalive` --- ### 📝 Commits (1) - [`7c315bf`](https://github.com/cbeuw/Cloak/commit/7c315bf7dad5bba20d23b045591b9f259bf48912) Make keepalive optional on client -> server and server -> proxy connections. Use KeepAlive value in config (seconds). ### 📊 Changes **4 files changed** (+22 additions, -6 deletions) <details> <summary>View changed files</summary> 📝 `cmd/ck-client/ck-client.go` (+2 -2) 📝 `cmd/ck-server/ck-server.go` (+2 -1) 📝 `internal/client/state.go` (+7 -0) 📝 `internal/server/state.go` (+11 -3) </details> ### 📄 Description Hello, This pull request just makes the default tcp keepalive since GO 1.13 of 15 seconds explicit. It is enabled on the client app, in the client cloak --> remote cloak direction and it is enabled on the server app in the server cloak --> server proxy direction. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 12:34:25 +03:00
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/Cloak#262
No description provided.