Skip to content

[FIX]: Bind GUI Compose publication to localhost - #2963

Open
Nina Chikanov (nina-msft) wants to merge 6 commits into
microsoft:mainfrom
nina-msft:fix/gui-compose-loopback
Open

Nina Chikanov (nina-msft) wants to merge 6 commits into
microsoft:mainfrom
nina-msft:fix/gui-compose-loopback

Conversation

@nina-msft

@nina-msft Nina Chikanov (nina-msft) commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

Description

Bind the GUI Docker Compose port to 127.0.0.1:8000:8000 instead of all host interfaces, matching the existing Jupyter default.

  • Preserve local GUI access, the internal container listener, and existing authentication behavior.
  • Add regression tests and a CI check for loopback-only GUI and Jupyter port mappings.
  • Document deployment precautions, container recreation, and local/remote verification.

Existing installations must recreate the GUI container to apply the new mapping; restarting it is not sufficient. This fixes default Docker publication, not authentication policy.

Tests and Documentation

  • 38 focused tests passed, covering Compose mappings and existing authentication behavior.
  • The resolved-Compose CI check passed locally, including after rebase.
  • Live checks verified loopback-only publication and working local GUI access before rebase.
  • Updated the Docker quick start and README with security and upgrade guidance.

Restrict the local GUI publication to loopback, guard resolved Compose bindings in CI, and document container recreation and verification.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@nina-msft Nina Chikanov (nina-msft) changed the title [DRAFT] [FIX]: Bind GUI Compose publication to localhost [FIX]: Bind GUI Compose publication to localhost Oct 6, 2026
@nina-msft
Nina Chikanov (nina-msft) marked this pull request as ready for review October 6, 2026 19:28
Comment thread docker/QUICKSTART.md
Comment thread tests/unit/docker/test_docker_compose.py
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

This branch has not been deployed

No deployments
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.

2 participants