[PR #2106] [MERGED] fix(Dockerfile): remove hardcoded exposed port #3888

Closed
opened 2026-02-22 18:34:42 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/2106
Author: @kabaluyot
Created: 8/13/2024
Status: Merged
Merged: 8/13/2024
Merged by: @timothycarambat

Base: masterHead: master


📝 Commits (1)

  • 7909056 build: remove hardcoded exposed port

📊 Changes

1 file changed (+0 additions, -3 deletions)

View changed files

📝 docker/Dockerfile (+0 -3)

📄 Description

Pull Request Type

  • feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

Exposed hardcoded port are causing "overriding" problems in Docker compose labels. In our use case, we are using traefik labels but the exposed port keeps overriding.

What is in this change?

Remove exposed port. It might also be improper since we can override th SERVER_PORT making the expose port not usable

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

🔄 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/Mintplex-Labs/anything-llm/pull/2106 **Author:** [@kabaluyot](https://github.com/kabaluyot) **Created:** 8/13/2024 **Status:** ✅ Merged **Merged:** 8/13/2024 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`7909056`](https://github.com/Mintplex-Labs/anything-llm/commit/7909056f42bdb3b99f8b8cdd9e8627bca580cddc) build: remove hardcoded exposed port ### 📊 Changes **1 file changed** (+0 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `docker/Dockerfile` (+0 -3) </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [ ] ✨ feat - [x] 🐛 fix - [ ] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues Exposed hardcoded port are causing "overriding" problems in Docker compose labels. In our use case, we are using traefik labels but the exposed port keeps overriding. ### What is in this change? Remove exposed port. It might also be improper since we can override th SERVER_PORT making the expose port not usable ### Developer Validations <!-- All of the applicable items should be checked. --> - [x] I ran `yarn lint` from the root of the repo & committed changes - [x] Relevant documentation has been updated - [x] I have tested my code functionality - [x] Docker build succeeds locally --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:34:42 -05:00
yindo closed this issue 2026-02-22 18:34:42 -05:00
yindo changed title from [PR #2106] fix(Dockerfile): remove hardcoded exposed port to [PR #2106] [MERGED] fix(Dockerfile): remove hardcoded exposed port 2026-06-05 15:15:44 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#3888