Skip to content

docs: add AllowTcpForwarding requirement to Hetzner SSH tunnel guide#54954

Closed
Aftabbs wants to merge 1 commit into
openclaw:mainfrom
Aftabbs:fix/hetzner-ssh-tunnel-tcp-forwarding
Closed

docs: add AllowTcpForwarding requirement to Hetzner SSH tunnel guide#54954
Aftabbs wants to merge 1 commit into
openclaw:mainfrom
Aftabbs:fix/hetzner-ssh-tunnel-tcp-forwarding

Conversation

@Aftabbs

@Aftabbs Aftabbs commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

Closes #54557

Problem

The Hetzner SSH tunnel setup silently fails when AllowTcpForwarding is disabled on the server (which is the default on many Hetzner configurations). Users get no error message, making this hard to diagnose.

Fix

Added a note to the Hetzner deployment guide explaining that AllowTcpForwarding yes must be set in /etc/ssh/sshd_config on the remote server for SSH tunnels to work.

@openclaw-barnacle openclaw-barnacle Bot added docs Improvements or additions to documentation size: XS labels Mar 26, 2026
@greptile-apps

greptile-apps Bot commented Mar 26, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR adds a small but useful diagnostic note to both the English (docs/install/hetzner.md) and Simplified Chinese (docs/zh-CN/install/hetzner.md) Hetzner deployment guides, explaining that SSH local-port-forwarding requires AllowTcpForwarding yes in the server's /etc/ssh/sshd_config. The note is well-placed immediately before the ssh -N -L … command, explains the symptom (tunnel connects but port is unreachable), and provides the remediation step (systemctl restart sshd). The Chinese translation accurately reflects the English note.

  • Documentation-only change; no code is modified.
  • Technically accurate: local TCP port forwarding is gated by AllowTcpForwarding in OpenSSH.
  • Both language variants are updated consistently.
  • No issues found.

Confidence Score: 5/5

Documentation-only fix with accurate technical content; safe to merge as-is.

Both changed files contain only a single-sentence note with correct technical information, proper placement, and a matching translation. There are no logic, syntax, or style concerns.

No files require special attention.

Important Files Changed

Filename Overview
docs/install/hetzner.md Added a note before the SSH tunnel command explaining the AllowTcpForwarding requirement.
docs/zh-CN/install/hetzner.md Added the Chinese translation of the same AllowTcpForwarding note, consistent with the English version.

Reviews (1): Last reviewed commit: "docs: add AllowTcpForwarding requirement..." | Re-trigger Greptile

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: e911158acf

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".


从你的笔记本电脑:

> **注意:** SSH 隧道要求服务器的 `/etc/ssh/sshd_config` 中设置 `AllowTcpForwarding yes`。Hetzner 的默认镜像可能已禁用此选项。如果隧道已连接但转发端口不可达,请检查此设置并在更改后重启 sshd(`systemctl restart sshd`)。

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Revert manual edit in generated zh-CN doc

This line introduces a direct edit under docs/zh-CN/**, but AGENTS.md explicitly marks that tree as generated and requires the i18n pipeline flow (update English docs, then regenerate translations) instead of manual changes. Keeping this hand-edited line risks it being overwritten on the next scripts/docs-i18n run and creates avoidable drift between source docs and generated translations, so this should be regenerated through the documented pipeline rather than committed directly.

Useful? React with 👍 / 👎.

@clawsweeper

clawsweeper Bot commented Apr 26, 2026

Copy link
Copy Markdown
Contributor

Closing this as duplicate or superseded after Codex automated review.

Close PR #54954 as duplicate/superseded by open PR #54564. Current main still lacks the Hetzner AllowTcpForwarding note, so this is not implemented-on-main; however #54564 already tracks the same #54557 docs gap with an English-only patch, while #54954 also edits generated localized docs and is currently dirty.

Best possible solution:

Close #54954 as a duplicate/superseded PR and continue the Hetzner SSH tunnel documentation fix in #54564. The best final patch should update only docs/install/hetzner.md, recommend the safer AllowTcpForwarding local setting for ssh -L, and let the docs i18n pipeline regenerate localized output.

What I checked:

So I’m closing this here and keeping the remaining discussion on the canonical linked item.

Codex Review notes: model gpt-5.5, reasoning high; reviewed against 06d409dc2738.

@clawsweeper clawsweeper Bot closed this Apr 26, 2026
vincentkoc pushed a commit that referenced this pull request Apr 29, 2026
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 9, 2026
github-actions Bot pushed a commit to Desicool/openclaw that referenced this pull request May 24, 2026
jameslcowan pushed a commit to jameslcowan/openclaw that referenced this pull request Jun 2, 2026
sablehead pushed a commit to sablehead/openclaw that referenced this pull request Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Improvements or additions to documentation size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Hetzner guide: SSH tunnel requires AllowTcpForwarding to be enabled in sshd config

1 participant