github / github/elastomer-client

Allow apps to configure faraday logger

Đang mở
#80 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Ruby
Star
197
Fork
28
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

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.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

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.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
elasticsearch, ruby
Lĩnh vực
backend-api-design, search
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
45/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.