github / github/elastomer-client

Allow apps to configure faraday logger

Aperta
#80 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Lingua principale
Ruby
Stelle
197
Fork
28
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

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.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
elasticsearch, ruby
Ambito
backend-api-design, search
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.