emit() does not call handleError on exception.

オープン
#172 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
48/100
issue の種類
バグ
明瞭さ
明確に書かれている
活発さ
停滞
技術スタック
python
領域
observability

調査の方向性

fluent/handler.py の237行目付近から始め、Handler の emit の動作を Python logging.Handler.handleError のドキュメントと比較してください。フォーマットまたはロギング中に発生する例外を追跡し、その後、fluent logger が例外を伝播させるのではなく handleError を呼び出すことを確認してください。

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

説明

bug

The documentation for the handleError function of logging.Handler implies that exceptions should be caught by emit and handled by invoking handleError. This allows the environment to customer how logging errors are handled.

The fluent logger Handler does not do this: https://github.com/fluent/fluent-logger-python/blob/d1b81bafe442f9cd3059fa07da1fd8de4e92716c/fluent/handler.py#L237

The net result is that exceptions triggered when running the fluent logger (either in format or otherwise) are raised, as opposed to log, which is inconsistent with the rest of the logging Handlers.

The fluent logger should call handleError.

主要言語
Python
スター
457
フォーク
138
30日以内にマージされた PR はありません

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

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

fluent/fluent-logger-python のほかの issue

fluent/fluent-logger-python の issue をすべて見る

似ている issue

Python の issue をもっと見る

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

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