shouldLogRequestHeader and shouldLogResponseHeader is not enough to control headers log in FULL mode

Aperta
  • Qualcosa non va?
#2,586 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
45/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
java
Ambito
api

Direzione di ricerca

Inizia in core/src/main/java/feign/Logger.java, alla riga di logging della risposta collegata nell’issue, e segui come vengono attualmente filtrati gli header della risposta. La modifica è completa quando gli header inclusi nel log del corpo della risposta in modalità FULL rispettano anch’essi shouldLogResponseHeader, impedendo che determinati header sensibili vi compaiano.

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

Descrizione

There is an implementation to control request and response headers with shouldLogRequestHeader and shouldLogResponseHeader.

It's working for logs but …

There is a line in the implementation which logs the whole response body also with headers.

I would also like to control these headers. Filter out particular headers in that particular log.

I am using FULL logging and debug mode.

The reason why I filter out the headers is to avoid security sensitive headers like tokens etc.

I would like to see that shouldLogResponseHeader is controlling also this line of the log too without any other implementation.

Here is how the log looks like

-10-04T10:36:50.206+02:00 DEBUG 131409 --- [demo] [           main] feign.Logger                             : [Client#get] {

  "args": {},

  "headers": {

    "Accept": "*/*",

    "Host": "[httpbin.org](http://httpbin.org/)",

    "User-Agent": "Java/21.0.1",

    "X-Amzn-Trace-Id": "Root=1-66ffa922-324f4c83482b70b4579853ac"

  },

  "origin": "88.212.19.236",

  "url": "https://httpbin.org/get"
}
Lingua principale
Java
Stelle
9.8k
Fork
2k
Merge medio
1h 15m
PR unite (30g)
39

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di OpenFeign/feign

Tutte le issue di OpenFeign/feign

Issue simili

Altre issue su Java

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.