Skip to content

[sshd] Add GatewayPorts option - #1464

Merged
Álvaro Rausell Guiard (AlvaroRausell) merged 1 commit into
devcontainers:mainfrom
lzhou-lilly:lingyu-sshd-gateway-ports
Sep 16, 2025
Merged

[sshd] Add GatewayPorts option#1464
Álvaro Rausell Guiard (AlvaroRausell) merged 1 commit into
devcontainers:mainfrom
lzhou-lilly:lingyu-sshd-gateway-ports

Conversation

@lzhou-lilly

@lzhou-lilly Lingyu Zhou (lzhou-lilly) commented Sep 3, 2025

Copy link
Copy Markdown
Contributor

We want to optionlly enable the GatewayPorts option in sshd.

To test: devcontainer features test --features sshd

Background:
Our development setup runs in Codespaces with Docker Compose. To handle TLS termination and spoof internal IPs, we use a reverse tunnel via gh cs ssh -- -R and proxys on workstations.

Without GatewayPorts, only the main container can access the tunnel—other containers can’t. Enabling GatewayPorts allows the tunnel to bind to the container’s public interfaces too, so other containers can use it. This is required for our transparent proxy setup.

@lzhou-lilly
Lingyu Zhou (lzhou-lilly) requested a review from a team as a code owner September 3, 2025 14:29
@lzhou-lilly

Lingyu Zhou (lzhou-lilly) commented Sep 3, 2025

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

This is a trivial work I initiated and completed on my own accord.

@lzhou-lilly Lingyu Zhou (lzhou-lilly) changed the title Add GatewayPorts option to sshd [sshd] Add GatewayPorts option Sep 4, 2025

@Kaniska244 Kaniska (Kaniska244) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi Lingyu Zhou (@lzhou-lilly)

Thank you for the contribution. The change looks fine. To be further reviewed by maintainers.

@AlvaroRausell
Álvaro Rausell Guiard (AlvaroRausell) merged commit 6a8863b into devcontainers:main Sep 16, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants