github / github/elastomer-client

Logging

Aperta
#5 6 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

We need to add some logging to the `Client` class. For each request the client makes, we should output a detailed debug message to a logger of some sort. I really like how Tire does it; logging a curl command that you can copy/paste and use directly. Also, outputting the status code of the response would be good, too ...

```
[200] curl -XGET 'http://localhost:9200/index/_search' -d '...'
```

Not sure how to handle large bodies and what not. And I don't think we need to log responses.

cc @github/search

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Start by locating the Client class and tracing how it makes requests and receives responses. Review the project's existing logger conventions, then determine how to represent the request as a copyable curl command, handle large bodies, and include the response status without logging responses. Done means each request produces the agreed debug output with suitable tests.

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

Valutazione

Stack tecnologico
elasticsearch, ruby
Ambito
api, observability
Tipo di issue
Funzionalità
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.