Skip to content

Add DOCUMENT_DOWNLOAD_API_HOST_NAME_INTERNAL to the Dockerfile#5965

Open
spatel033 wants to merge 1 commit into
mainfrom
add-document-download-api-host-name
Open

Add DOCUMENT_DOWNLOAD_API_HOST_NAME_INTERNAL to the Dockerfile#5965
spatel033 wants to merge 1 commit into
mainfrom
add-document-download-api-host-name

Conversation

@spatel033

@spatel033 spatel033 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Add missing DOCUMENT_DOWNLOAD_API_HOST_NAME_INTERNAL to the dockerfile

@quis quis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you also need DOCUMENT_DOWNLOAD_API_HOST, per:

DOCUMENT_DOWNLOAD_API_HOST = os.environ.get("DOCUMENT_DOWNLOAD_API_HOST", "http://localhost:7000")
DOCUMENT_DOWNLOAD_API_HOST_NAME_INTERNAL = os.environ.get(
"DOCUMENT_DOWNLOAD_API_HOST_NAME_INTERNAL", "http://localhost:7000"
)

?

Or if that variable is unused, can we delete it?

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.

2 participants