litellm icon indicating copy to clipboard operation
litellm copied to clipboard

feat: Add support for file-based secrets in config values

Open bexelbie opened this issue 8 months ago • 3 comments

Title

Implement reading values (api_key, etc.) from files.

Relevant issues

Fixes #9240

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

  • [X] I have Added testing in the tests/litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • [X] I have added a screenshot of my new test passing locally
  • [X] My PR passes all unit tests on (make test-unit)[https://docs.litellm.ai/docs/extras/contributing_code]
  • [X] My PR's scope is as isolated as possible, it only solves 1 specific problem

Type

🆕 New Feature 📖 Documentation ✅ Test

Changes

Added file/ as a valid config file marker for read from file similar to os.environ/ and oidc/ Tests added Docs added

Note: Linting and unit test failures unrelated to my code have been ignored.

shot

bexelbie avatar Mar 14 '25 15:03 bexelbie

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
litellm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 14, 2025 3:24pm

vercel[bot] avatar Mar 14 '25 15:03 vercel[bot]

Let me know if this can be merged and I'll resolve the conflicts. Thank you

bexelbie avatar Apr 02 '25 03:04 bexelbie

CLA assistant check
All committers have signed the CLA.

CLAassistant avatar Apr 22 '25 22:04 CLAassistant

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

github-actions[bot] avatar Jul 24 '25 00:07 github-actions[bot]