github / github/elastomer-client

Allow apps to configure faraday logger

オープン
#80 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
Ruby
スター
197
フォーク
28
PR マージ指標
30日以内にマージされた PR はありません

説明

This code in haystack reaches into elastomer-client's Faraday object to set the logger:

```
if Rails.env.development?
Haystack.elastomer_client.connection.response :logger, Rails.logger
end
```

https://github.com/github/haystack/blob/master/config/initializers/elasticsearch.rb#L18-L20

This shouldn't be necessary. Elastomer::Client should provide a method to set a logger object.

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

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

調査の方向性

Start with Elastomer::Client and the connection setup used by the Haystack initializer at config/initializers/elasticsearch.rb#L18-L20. Trace how the Faraday connection is exposed and determine where a logger-setting method belongs. Done means an app can configure the logger through Elastomer::Client without reaching into the Faraday object.

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

評価

技術スタック
elasticsearch, ruby
領域
backend-api-design, search
issue の種類
機能追加
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

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

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