[PR #24335] [MERGED] fix(docs): server and machine-learning use IMMICH_HOST and IMMICH_PORT #17828

Closed
opened 2026-02-05 16:28:13 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/immich-app/immich/pull/24335
Author: @dionysius
Created: 12/2/2025
Status: Merged
Merged: 12/2/2025
Merged by: @jrasm91

Base: mainHead: docsmllistener


📝 Commits (1)

  • 8a00b2f fix(docs): server and machine-learning use IMMICH_HOST and IMMICH_PORT

📊 Changes

1 file changed (+4 additions, -4 deletions)

View changed files

📝 docs/docs/install/environment-variables.md (+4 -4)

📄 Description

Description

It was not clear which environment variables the machine-learning server uses to build its listener address.

Followup of #24304

How Has This Been Tested?

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/)

Please describe to which degree, if any, an LLM was used in creating this pull request.

none


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/immich-app/immich/pull/24335 **Author:** [@dionysius](https://github.com/dionysius) **Created:** 12/2/2025 **Status:** ✅ Merged **Merged:** 12/2/2025 **Merged by:** [@jrasm91](https://github.com/jrasm91) **Base:** `main` ← **Head:** `docsmllistener` --- ### 📝 Commits (1) - [`8a00b2f`](https://github.com/immich-app/immich/commit/8a00b2f35bf626624d3a5b41f89c17f619f9c80c) fix(docs): server and machine-learning use IMMICH_HOST and IMMICH_PORT ### 📊 Changes **1 file changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `docs/docs/install/environment-variables.md` (+4 -4) </details> ### 📄 Description ## Description It was not clear which environment variables the machine-learning server uses to build its listener address. Followup of #24304 ## How Has This Been Tested? - ## Checklist: - [x] I have performed a self-review of my own code - [x] I have made corresponding changes to the documentation if applicable - [x] 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/`) ## Please describe to which degree, if any, an LLM was used in creating this pull request. none --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-05 16:28:13 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: immich-app/immich#17828