Skip to content

feat: add log statement indicating config source#18

Merged
lelia merged 1 commit into
mainfrom
feature/log-config-source
Jun 26, 2026
Merged

feat: add log statement indicating config source#18
lelia merged 1 commit into
mainfrom
feature/log-config-source

Conversation

@dc-larsen

Copy link
Copy Markdown
Contributor

Summary

  • Adds an INFO-level log message that tells users where the configuration is being loaded from
  • Tracks the config source internally via a _config_source key
  • Outputs user-friendly descriptions for each source type

Config Sources

The three possible sources (in priority order) are:

  1. Socket dashboard (API) - when using enterprise plan with dashboard config
  2. JSON config file (--config) - when a config file is specified via CLI
  3. Environment variables - the default when no other source is available

Example Output

2024-01-06 13:20:00 - INFO - Configuration loaded from: Socket dashboard (API)

or

2024-01-06 13:20:00 - INFO - Configuration loaded from: environment variables

Test plan

  • Run socket-basics with no config (should show "environment variables")
  • Run socket-basics with --config config.json (should show "JSON config file")
  • Run socket-basics with API key and enterprise org (should show "Socket dashboard (API)")

🤖 Generated with Claude Code

@dc-larsen David Larsen (dc-larsen) requested a review from a team as a code owner January 6, 2026 21:34
@lelia lelia requested a review from a team as a code owner March 3, 2026 00:09
@lelia lelia self-assigned this Jun 26, 2026
@lelia lelia changed the title Add log statement indicating config source feat: add log statement indicating config source Jun 26, 2026
@lelia lelia force-pushed the feature/log-config-source branch from 91ff15c to a550947 Compare June 26, 2026 21:47
Log the selected configuration source during Config initialization.
@lelia lelia force-pushed the feature/log-config-source branch from a550947 to e188d70 Compare June 26, 2026 21:50
@lelia lelia merged commit 8998fc9 into main Jun 26, 2026
11 checks passed
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