Skip to content

Conversation

@arealmaas
Copy link
Collaborator

Description

Ref discussion on Slack: https://digdir.slack.com/archives/C07R312076E/p1764159034252739

Setting log level to info on logs for npsql

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 26, 2025

📝 Walkthrough

Walkthrough

Two appsettings JSON files were updated to add a Serilog configuration block that sets MinimumLevel.Override for Npgsql and Npgsql.Internal to "Information". No other configuration or public API changes were made.

Changes

Cohort / File(s) Summary
Serilog overrides added
src/Digdir.Domain.Dialogporten.GraphQL/appsettings.yt01.json, src/Digdir.Domain.Dialogporten.WebApi/appsettings.yt01.json
Added a Serilog section including MinimumLevel.Override entries: "Npgsql": "Information" and "Npgsql.Internal": "Information". No other changes to these files' settings.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Identical, repetitive configuration additions across two files — low complexity.
  • Quick checks: ensure the Serilog block is correctly merged into existing JSON structure and values are intended for the targeted environment.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description references a Slack discussion and explains the change clearly, but is missing several template sections including Verification checklist and Documentation status.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title 'chore: enable info logs on npsql' directly describes the main change: adding Serilog configuration to enable information-level logs for Npgsql across two appsettings files.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/enable-info-logs-on-npsql

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link

@arealmaas arealmaas changed the title feat: enable info logs on npsql chore: enable info logs on npsql Nov 26, 2025
@arealmaas arealmaas merged commit 18dbe97 into main Nov 26, 2025
31 checks passed
@arealmaas arealmaas deleted the feat/enable-info-logs-on-npsql branch November 26, 2025 12:59
arealmaas pushed a commit that referenced this pull request Nov 26, 2025
🤖 I have created a release *beep* *boop*
---


##
[1.92.1](v1.92.0...v1.92.1)
(2025-11-26)


### Miscellaneous Chores

* enable info logs on npsql
([#3059](#3059))
([18dbe97](18dbe97))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

2 participants