labthings / labthings/labthings-fastapi

Change Logging Level from a Config File

Open
#286 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Python
Stars
9
Forks
4
PR merge metrics
No merged PRs in 30d

Description

Is your feature request related to a problem? Please describe.
Following on from #282 - we can now set the LabThings logging level for debug using the cli. As @rwb27 notes, in the future it may make sense to put debug into the config file as well as having it in cli.

Describe the solution you'd like
Add debug to config file options, using 'from_config()'

Describe alternatives you've considered
We currrently have a cli flag. This would be retained in addition to adding debug in the config file.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by tracing the existing CLI debug logging option and the config loader's from_config() path. Identify how logging settings are represented and applied, then retain the CLI behavior while accepting debug from the config file. Done when a config containing debug changes the logging level and the existing CLI flag still works.

Written by the indexing model from the issue text.

Assessment

Tech stack
fastapi, python
Domain
backend
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
65/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.