Skip to content

fix: Remove hardcoded PostgreSQL password from docker-compose.yml (se… - #979

Open
ford220102 wants to merge 4 commits into
IceWhaleTech:mainfrom
ford220102:main
Open

fix: Remove hardcoded PostgreSQL password from docker-compose.yml (se…#979
ford220102 wants to merge 4 commits into
IceWhaleTech:mainfrom
ford220102:main

Conversation

@ford220102

Copy link
Copy Markdown

…crets:S6698)

Security fix for secrets:S6698 - PostgreSQL database passwords should not be disclosed.

Removed hardcoded password "v8k2mxpq93nw5ztl" from docker-compose.yml and replaced with environment variables.

Changes:

  • DATABASE_URL now uses ${POSTGRES_USER}, ${POSTGRES_PASSWORD}, ${POSTGRES_DB}
  • POSTGRES_PASSWORD uses ${POSTGRES_PASSWORD} (no default - must be set)
  • POSTGRES_USER uses ${POSTGRES_USER:-dokploy} with default fallback
  • POSTGRES_DB uses ${POSTGRES_DB:-dokploy} with default fallback
  • Updated description for POSTGRES_PASSWORD to instruct users to generate a secure random password
  • Added tip about generating password to before_install section
  • Added .env.example file with instructions

This prevents credentials from being exposed in the repository and ensures each installation uses a unique, secure password.

How to use:

  1. Create .env file: cp .env.example .env
  2. Generate secure password: openssl rand -base64 32
  3. Set POSTGRES_PASSWORD in .env file
  4. Run docker-compose up -d

…crets:S6698)

Security fix for secrets:S6698 - PostgreSQL database passwords should
not be disclosed.

Removed hardcoded password "v8k2mxpq93nw5ztl" from docker-compose.yml
and replaced with environment variables.

Changes:
- DATABASE_URL now uses ${POSTGRES_USER}, ${POSTGRES_PASSWORD}, ${POSTGRES_DB}
- POSTGRES_PASSWORD uses ${POSTGRES_PASSWORD} (no default - must be set)
- POSTGRES_USER uses ${POSTGRES_USER:-dokploy} with default fallback
- POSTGRES_DB uses ${POSTGRES_DB:-dokploy} with default fallback
- Updated description for POSTGRES_PASSWORD to instruct users to generate
  a secure random password
- Added tip about generating password to before_install section
- Added .env.example file with instructions

This prevents credentials from being exposed in the repository and
ensures each installation uses a unique, secure password.

How to use:
1. Create .env file: cp .env.example .env
2. Generate secure password: openssl rand -base64 32
3. Set POSTGRES_PASSWORD in .env file
4. Run docker-compose up -d
…mpatible

Security fix for secrets:S6698 - PostgreSQL database passwords should
not be disclosed.

Removed hardcoded password "v8k2mxpq93nw5ztl" from docker-compose.yml
and replaced with environment variables accessible via CasaOS UI.

Changes:
- DATABASE_URL uses ${POSTGRES_PASSWORD} variable
- POSTGRES_PASSWORD uses ${POSTGRES_PASSWORD:-dokploy_secure_password_change_me}
- Added POSTGRES_PASSWORD to x-casaos.envs with description to change it
- Added tip in before_install about changing the password
- Users can now set/change password through CasaOS installation dialog

This prevents credentials from being exposed in the repository and
allows users to set their own secure password during installation.
Fixed icon and screenshot URLs in docker-compose.yml to point to the
fork repository (ford220102/CasaOS-AppStore) instead of the upstream
repository (IceWhaleTech/CasaOS-AppStore).

This resolves the build error:
"ERROR App 'org.icewhale.dokploy' failed: App 'Dokploy' failed to
resolve icon"

Changes:
- icon: Updated path to use ford220102/CasaOS-AppStore
- screenshot_link: Updated all three screenshot paths to use
  ford220102/CasaOS-AppStore

This ensures the build can find the icon.svg and screenshot files
during the validation process.
@sonarqubecloud

Copy link
Copy Markdown

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.

1 participant