Fix hardcoded localhost URL in FileURLService breaking Docker deployment #26
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
FileURLServicehardcodedhttp://localhost:8000as defaultbase_url, causing remote MinerU API to fail in Docker environments where localhost is unreachable.Changes
FileURLService.__init__: Remove hardcoded default, requireFILE_SERVICE_BASE_URLenv varget_file_service(): Delegate env var handling to__init__instead of duplicating logicenv.example: UpdateFILE_SERVICE_BASE_URLwith clear "must configure" warning and placeholder valuehttp://your-ip:8000instead of localhostBehavior
Explicit
base_urlparameter still works for testing/override scenarios.Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.