github / github/elastomer-client
Allow apps to configure faraday logger
- 主要言語
- 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