Skip to content

feat: allow configurable collector port#5607

Merged
timothycarambat merged 5 commits into
Mintplex-Labs:masterfrom
officialasishkumar:5114-configurable-collector-port
May 12, 2026
Merged

feat: allow configurable collector port#5607
timothycarambat merged 5 commits into
Mintplex-Labs:masterfrom
officialasishkumar:5114-configurable-collector-port

Conversation

@officialasishkumar
Copy link
Copy Markdown
Contributor

Pull Request Type

  • ✨ feat (New feature)
  • 🐛 fix (Bug fix)
  • ♻️ refactor (Code refactoring without changing behavior)
  • 💄 style (UI style changes)
  • 🔨 chore (Build, CI, maintenance)
  • 📝 docs (Documentation updates)

Relevant Issues

resolves #5114

Description

Allows the document collector process to listen on a configured port using COLLECTOR_PORT, instead of always binding to 8888.

The server-side collector client already reads COLLECTOR_PORT; this change makes the collector process use the same setting, applies the same port validation on the server endpoint, and documents the variable in the server, collector, and Docker env examples.

Visuals (if applicable)

N/A

Additional Information

Invalid COLLECTOR_PORT values fall back to 8888, matching the existing default behavior.

Developer Validations

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

Validation performed:

  • node --check collector/index.js
  • node --check server/utils/collectorApi/index.js
  • node --check server/utils/comKey/index.js
  • YARN_CACHE_FOLDER=.yarn-cache yarn install --frozen-lockfile --ignore-scripts in collector
  • YARN_CACHE_FOLDER=.yarn-cache yarn install --frozen-lockfile --ignore-scripts in server
  • yarn lint:check in collector
  • yarn lint:check in server
  • git diff --check

@timothycarambat timothycarambat added the PR: Ready for QA Indicates that a PR is ready for QA and the GH image may build the image label May 11, 2026
@timothycarambat timothycarambat self-assigned this May 11, 2026
@timothycarambat timothycarambat removed the PR: Ready for QA Indicates that a PR is ready for QA and the GH image may build the image label May 12, 2026
@timothycarambat
Copy link
Copy Markdown
Member

export STORAGE_LOCATION=$HOME/anythingllm && \
mkdir -p $STORAGE_LOCATION && \
touch "$STORAGE_LOCATION/.env" && \
docker run -d -p 3001:3001 \
--cap-add SYS_ADMIN \
-v ${STORAGE_LOCATION}:/app/server/storage \
-v ${STORAGE_LOCATION}/.env:/app/server/.env \
-e STORAGE_DIR="/app/server/storage" \
-e COLLECTOR_PORT=9999 \ # This will set the param for both now
<image>

@timothycarambat timothycarambat merged commit 720ea69 into Mintplex-Labs:master May 12, 2026
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.

[FEAT]: Ability to lock listening port/change it away from 8888

2 participants