Skip to content

Fix/landing page availability#14

Merged
zierh merged 4 commits into
mainfrom
fix/landing-page-availability
May 5, 2026
Merged

Fix/landing page availability#14
zierh merged 4 commits into
mainfrom
fix/landing-page-availability

Conversation

@zierh
Copy link
Copy Markdown
Collaborator

@zierh zierh commented May 4, 2026

details: see commit messages

zierh added 2 commits May 4, 2026 10:59
landing page was not reachable with network mode host
the docker socket under /var/run was not the one beeing used in NixOS.
This prevented the use of docker to start the open-wrt container for the
'Time Travelers Update'-CTF challenge from inside the landing page
container.
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adjusts the devcontainer “virtual” docker-compose setup to improve the availability/behavior of the landing service by changing how it connects to the host and Docker daemon.

Changes:

  • Removed network_mode: host from the landing service and kept an explicit 80:80 port mapping.
  • Switched the landing service’s Docker socket bind mount from /var/run/docker.sock to ${XDG_RUNTIME_DIR}/docker.sock.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .devcontainer/virtual/docker-compose.yml
Comment thread .devcontainer/virtual/docker-compose.yml Outdated
zierh and others added 2 commits May 4, 2026 11:09
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Comment thread .devcontainer/virtual/docker-compose.yml Outdated
@zierh zierh merged commit d30f332 into main May 5, 2026
6 of 11 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