Skip to content

Comments

Set configurable logging levels and improve log clarity#122

Merged
cgalibern merged 3 commits intoopensvc:mainfrom
cgalibern:dev
Feb 22, 2026
Merged

Set configurable logging levels and improve log clarity#122
cgalibern merged 3 commits intoopensvc:mainfrom
cgalibern:dev

Conversation

@cgalibern
Copy link
Contributor

@cgalibern cgalibern commented Feb 22, 2026 

### Description

This pull request introduces improvements to logging capabilities:

- **Set default log level to `Debug`**: Automatically enabled when the `debug` flag is used.
- **Log request level configuration**: Replaces the `log.request` flag with `log.request.level` to support configurable log levels (e.g., `none`, `debug`, `info`). Middleware updated to accommodate this change.
- **Improved logging clarity**: Replaced URI logging with URIPath to enhance field clarity.

@cgalibern
[logging] Log URIPath instead of URI
59823a3 
- Replaced the `LogURI` field with `LogURIPath` for better naming clarity.
@cgalibern
[cmd] Update log request configuration to support log levels (log.req…
51f38a9 
…uest.level:none)

- Replaced the `log.request` boolean flag with `log.request.level` for configurable log levels.
- Updated `LogRequestMiddleware` to handle dynamic log levels.
- Adjusted default configurations to use `log.request.level` with "none" as the default value.
@cgalibern
[cmd] Set default log level to Debug when debug flag is enabled
c746a0f
@cgalibern cgalibern merged commit 3937661 into opensvc:main Feb 22, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@cgalibern