docs: add AllowTcpForwarding requirement to Hetzner SSH tunnel guide#54954
docs: add AllowTcpForwarding requirement to Hetzner SSH tunnel guide#54954Aftabbs wants to merge 1 commit into
Conversation
Greptile SummaryThis PR adds a small but useful diagnostic note to both the English (
Confidence Score: 5/5Documentation-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.
|
| 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
There was a problem hiding this comment.
💡 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`)。 |
There was a problem hiding this comment.
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 👍 / 👎.
|
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 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. |
Fixes #54557; carries forward #54564; refs #54954. Thanks @satishkc7, @blackstrype, and @Aftabbs.
Fixes openclaw#54557; carries forward openclaw#54564; refs openclaw#54954. Thanks @satishkc7, @blackstrype, and @Aftabbs.
Fixes openclaw#54557; carries forward openclaw#54564; refs openclaw#54954. Thanks @satishkc7, @blackstrype, and @Aftabbs.
Fixes openclaw#54557; carries forward openclaw#54564; refs openclaw#54954. Thanks @satishkc7, @blackstrype, and @Aftabbs.
Fixes openclaw#54557; carries forward openclaw#54564; refs openclaw#54954. Thanks @satishkc7, @blackstrype, and @Aftabbs.
Closes #54557
Problem
The Hetzner SSH tunnel setup silently fails when
AllowTcpForwardingis 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 yesmust be set in/etc/ssh/sshd_configon the remote server for SSH tunnels to work.