Skip to content

Conversation

@X9X0
Copy link
Owner

@X9X0 X9X0 commented Nov 18, 2025

PROBLEM:

  • Security system environment variables were added to .env.example but not to docker-compose.yml
  • This caused the container to not receive LABLINK_ENABLE_ADVANCED_SECURITY=true
  • Result: Security manager wasn't initialized, login failed with 500 error

CHANGES:

  • Added LABLINK_ENABLE_ADVANCED_SECURITY to lablink-server environment
  • Added LABLINK_CREATE_DEFAULT_ADMIN to lablink-server environment
  • Added LABLINK_DEFAULT_ADMIN_USERNAME to lablink-server environment
  • Added LABLINK_DEFAULT_ADMIN_PASSWORD to lablink-server environment
  • Added LABLINK_DEFAULT_ADMIN_EMAIL to lablink-server environment

All variables default to values from .env file, with fallback defaults in docker-compose.yml to ensure the security system is always enabled.

This completes the security system initialization fix that was partially merged in the previous PR.

PROBLEM:
- Security system environment variables were added to .env.example but not
  to docker-compose.yml
- This caused the container to not receive LABLINK_ENABLE_ADVANCED_SECURITY=true
- Result: Security manager wasn't initialized, login failed with 500 error

CHANGES:
- Added LABLINK_ENABLE_ADVANCED_SECURITY to lablink-server environment
- Added LABLINK_CREATE_DEFAULT_ADMIN to lablink-server environment
- Added LABLINK_DEFAULT_ADMIN_USERNAME to lablink-server environment
- Added LABLINK_DEFAULT_ADMIN_PASSWORD to lablink-server environment
- Added LABLINK_DEFAULT_ADMIN_EMAIL to lablink-server environment

All variables default to values from .env file, with fallback defaults in
docker-compose.yml to ensure the security system is always enabled.

This completes the security system initialization fix that was partially
merged in the previous PR.
@X9X0 X9X0 merged commit 7a8c025 into main Nov 18, 2025
25 checks passed
@X9X0 X9X0 deleted the claude/raspberry-pi-image-builder-01NP4stSktZyRQfSmihnmwdw branch November 18, 2025 03:07
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