Azure / Azure/data-api-builder

[Improvement]: Enable logging based on the appsettings file

オープン
#1,815 コメント 1 件 リアクション 1 件 担当者 0 名 GitHub で見る
enhancement
主要言語
C#
スター
1.5k
フォーク
370
平均マージ
3日 22時間
マージ済み PR(30日)
9

説明

### What happened?

At this point only Application Insights
https://github.com/Azure/data-api-builder/blob/a0b64c0196c9f8ec722bafdb0d1e4c9bc8a1a016/src/Service/Program.cs#L131
and Console
https://github.com/Azure/data-api-builder/blob/a0b64c0196c9f8ec722bafdb0d1e4c9bc8a1a016/src/Service/Program.cs#L144
can be configured as the logger output.

As a user I would like to use the appsettings.json configuration to support other ways of preserving the log entries and their log levels.

### Version

0.8.52

### What database are you using?

Azure SQL

### What hosting model are you using?

Local (including CLI)

### Which API approach are you accessing DAB through?

REST

### Relevant log output

_No response_

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

src/Service/Program.cs のリンク先にある Application Insights と Console の構成箇所からロギング設定の確認を始め、次に appsettings.json がどのように読み込まれるかを調査します。追加で必要なロギング出力とログレベル設定を明確にします。合意したオプションを、現在サポートされている出力だけでなく appsettings.json からも構成できるようになれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
csharp
領域
observability
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。