[BUG] Docker Secrets are not supported for all sensitive environment variables #859

Closed
opened 2026-02-04 23:04:52 +03:00 by OVERLORD · 2 comments
Owner

Originally created by @mark-monteiro on GitHub (May 18, 2023).

The bug

Docker secret support (https://github.com/immich-app/immich/issues/14) was partially implemented in https://github.com/immich-app/immich/pull/1254, but only for PostgreSQL environment variables. There are a few other sensitive environment variables that would benefit from docker secret support that were missed in the original implementation:

  • TYPESENSE_API_KEY
  • REDIS_USERNAME
  • REDIS_PASSWORD
  • others?

The OS that Immich Server is running on

Docker (Ubuntu host)

Version of Immich Server

v1.55.1

Version of Immich Mobile App

v1.55.1

Platform with the issue

  • Server
  • Web
  • Mobile

Your docker-compose.yml content

N/A

Your .env content

N/A

Reproduction steps

N/A

Additional information

No response

Originally created by @mark-monteiro on GitHub (May 18, 2023). ### The bug Docker secret support (https://github.com/immich-app/immich/issues/14) was partially implemented in https://github.com/immich-app/immich/pull/1254, but only for PostgreSQL environment variables. There are a few other sensitive environment variables that would benefit from docker secret support that were missed in the original implementation: - TYPESENSE_API_KEY - REDIS_USERNAME - REDIS_PASSWORD - others? ### The OS that Immich Server is running on Docker (Ubuntu host) ### Version of Immich Server v1.55.1 ### Version of Immich Mobile App v1.55.1 ### Platform with the issue - [X] Server - [ ] Web - [ ] Mobile ### Your docker-compose.yml content ```YAML N/A ``` ### Your .env content ```Shell N/A ``` ### Reproduction steps ```bash N/A ``` ### Additional information _No response_
Author
Owner

@mark-monteiro commented on GitHub (May 18, 2023):

After a bit more investigation, neither Typesense nor Redis support Docker secrets in the compose file for the environment variables mentioned. Which makes supporting them in Immich a bit useless since they'll need to be exposed as environment variables for the other services anyways. I will close this issue.

@mark-monteiro commented on GitHub (May 18, 2023): After a bit more investigation, neither Typesense nor Redis support Docker secrets in the compose file for the environment variables mentioned. Which makes supporting them in Immich a bit useless since they'll need to be exposed as environment variables for the other services anyways. I will close this issue.
Author
Owner

@abhi-chakrab commented on GitHub (Jul 15, 2023):

@mark-monteiro - Is there a possibility to at least add the REDIS_PASSWORD as a secret. It is possible to use a workaround as per the below suggestion from the official repo:
https://github.com/docker-library/redis/issues/46#issuecomment-335326234

I have also created a PR for the basic script changes - https://github.com/immich-app/immich/pull/3282

@abhi-chakrab commented on GitHub (Jul 15, 2023): @mark-monteiro - Is there a possibility to at least add the REDIS_PASSWORD as a secret. It is possible to use a workaround as per the below suggestion from the official repo: https://github.com/docker-library/redis/issues/46#issuecomment-335326234 I have also created a PR for the basic script changes - https://github.com/immich-app/immich/pull/3282
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#859