Skip to content

Conversation

@amarshall
Copy link
Contributor

Description

See https://systemd.io/CREDENTIALS/. This is used as a fallback, so will only be used if the $*_FILE var is empty. This could also be used to implicitly use Docker Secrets by settings
CREDENTIALS_DIRECTORY=/run/secrets rather than setting individual $_*FILE environment variables.

How Has This Been Tested?

I patched Immich using this change and tested the functionality.

Checklist:

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if applicable
  • I have no unrelated changes in the PR.
  • I have confirmed that any new dependencies are strictly necessary.
  • I have written tests for new code (if applicable)
  • I have followed naming conventions/patterns in the surrounding code
  • All code in src/services/ uses repositories implementations for database calls, filesystem operations, etc.
  • All code in src/repositories/ is pretty basic/simple and does not have any immich specific logic (that belongs in src/services/)

See https://systemd.io/CREDENTIALS/. This is used as a fallback, so will
only be used if the `$*_FILE` var is empty. This could also be used to
implicitly use Docker Secrets by settings
`CREDENTIALS_DIRECTORY=/run/secrets` rather than setting individual
`$_*FILE` environment variables.
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Mar 29, 2025
@jrasm91
Copy link
Member

jrasm91 commented Jun 12, 2025

Can you rebase this against main please?

@danieldietzler
Copy link
Member

Hey @amarshall could you please rebase the PR or allow editing for contributors?

@jrasm91
Copy link
Member

jrasm91 commented Jun 23, 2025

I'm just going to close it. If they make the changes they can open it again.

@amarshall
Copy link
Contributor Author

Reopened as #20406

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:feature documentation Improvements or additions to documentation needs-answer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants