Skip to content

feat: add support for secret environment files#3

Open
hbenali wants to merge 1 commit intomasterfrom
secrets
Open

feat: add support for secret environment files#3
hbenali wants to merge 1 commit intomasterfrom
secrets

Conversation

@hbenali
Copy link
Contributor

@hbenali hbenali commented Oct 8, 2025

This commit introduces support for managing sensitive environment variables using secret files in the eXo Platform Docker image.

  • Environment variables prefixed with EXO_SEC_ and suffixed with _FILE are read from the specified file paths.
  • The file contents are exported as standard environment variables for use by the application.
  • Enables secure handling of passwords, API keys, and private keys without hardcoding them in Docker commands or Compose files.

Example usage:

  • EXO_SEC_DB_PASSWORD_FILE=/run/secrets/exo_db_password
  • EXO_SEC_MAIL_SMTP_PASSWORD_FILE=/run/secrets/exo_mail_smtp_password

This commit introduces support for managing sensitive environment variables
using secret files in the eXo Platform Docker image.

- Environment variables prefixed with EXO_SEC_ and suffixed with _FILE
  are read from the specified file paths.
- The file contents are exported as standard environment variables for
  use by the application.
- Enables secure handling of passwords, API keys, and private keys
  without hardcoding them in Docker commands or Compose files.

Example usage:
  EXO_SEC_DB_PASSWORD_FILE=/run/secrets/exo_db_password
  EXO_SEC_MAIL_SMTP_PASSWORD_FILE=/run/secrets/exo_mail_smtp_password
Copy link
Contributor

@bourasom bourasom left a comment

Choose a reason for hiding this comment

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

LGTM

@bourasom
Copy link
Contributor

if both envs (with SEC and without) are put, is there a priority? Shall We add test on that?

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

Comments