Skip to content

Added Docker secret support.#95

Open
mooseh wants to merge 7 commits intoBrettMayson:v2from
mooseh:v2
Open

Added Docker secret support.#95
mooseh wants to merge 7 commits intoBrettMayson:v2from
mooseh:v2

Conversation

@mooseh
Copy link
Copy Markdown

@mooseh mooseh commented Feb 12, 2026

Summary of work

  • added secret overrides when secret files exist
  • updated README
  • added break-system-packages to pip install command due to it failing.

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

Adds support for sourcing Steam credentials from Docker secrets at runtime, along with documentation updates and a Docker build tweak to allow pip installs on Debian Bookworm.

Changes:

  • Read /var/run/secrets/STEAM_USER and /var/run/secrets/STEAM_PASSWORD (when present) and apply them during container startup.
  • Document Docker secrets usage with a docker-compose example in the README.
  • Add --break-system-packages to the Dockerfile pip install step to avoid build failures.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
launch.py Adds Docker secrets file reads to override Steam credentials at startup.
README.md Documents Docker secrets support and provides a compose snippet.
Dockerfile Adjusts pip install command to work with Debian’s externally-managed Python environment.

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

Comment thread launch.py
Comment thread README.md Outdated
Comment thread README.md Outdated
mooseh and others added 2 commits February 19, 2026 08:21
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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