feat(cli): support MARKITDOWN_CU_ENDPOINT and MARKITDOWN_DOCINTEL_ENDPOINT env vars - #2330
Open
Rafli Surya Wijaya (loulanyue) wants to merge 1 commit into
Open
Conversation
…POINT env vars Allow Azure Content Understanding and Document Intelligence endpoints to be configured via MARKITDOWN_CU_ENDPOINT and MARKITDOWN_DOCINTEL_ENDPOINT environment variables as fallbacks when explicit CLI flags are omitted. Closes microsoft#2326
|
Rafli Surya Wijaya (@loulanyue) please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Adds environment-variable fallbacks for Azure Content Understanding and Document Intelligence endpoints in the CLI:
--cu-endpointfalls back toMARKITDOWN_CU_ENDPOINT-e/--endpointfalls back toMARKITDOWN_DOCINTEL_ENDPOINTFixes #2326
Rationale
Credentials (
AZURE_API_KEYorDefaultAzureCredential) are already ambiently resolved from the environment. Allowing endpoints to be configured via environment variables enables operator-configured deployments and clean tool invocations for automated agents and scripts without needing resource URLs hardcoded in command-line arguments.Explicit CLI flags (
--cu-endpoint,-e/--endpoint) take precedence over environment variables.Changes
packages/markitdown/src/markitdown/__main__.py:docintel_endpoint = args.endpoint or os.environ.get("MARKITDOWN_DOCINTEL_ENDPOINT").cu_endpoint = args.cu_endpoint or os.environ.get("MARKITDOWN_CU_ENDPOINT").packages/markitdown/tests/test_cli_misc.py:Verification
pytest packages/markitdown/tests/test_cli_misc.py -v: