spring-cloud/spring-cloud-gateway

Request headers can not be modified after request pass through ModifyRequestBodyGatewayFilter

Aperta

#2548 aperta il 11 mar 2022

Vedi su GitHubGuida contributor
 (0 commenti) (0 reazioni) (0 assegnatari)Java (3204 fork)batch import
documentationhelp wanted

Metriche repository

Star
 (4284 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

Describe the bug Spring cloud version:3.0.3 I am using ModifyRequestBodyGatewayFilter to cache request body, but when request pass through ModifyRequestBodyGatewayFilter, heads can not be modified any more, at opposite, headers can be modified before ModifyRequestBodyGatewayFilter.

I guess this problem happened because decorate method from ModifyRequestBodyGatewayFilterFactory. Spring cloud gateway user can not get new headers from MutatedServerHttpRequest but can only get header from the time point when decorate method be called. image

I am not sure this is an issue or it's a mistake about my usage.

Guida contributor

Direzione di ricerca
Indaga sul metodo `decorate` in `ModifyRequestBodyGatewayFilterFactory` per capire come vengono catturate le intestazioni. Controlla l'implementazione di `MutatedServerHttpRequest` per vedere se le modifiche alle intestazioni dopo la decorazione vengono ignorate. Cerca problemi o commit correlati nel repository che potrebbero riguardare l'ordine di modifica delle intestazioni. Considera se la correzione richiede di esporre le intestazioni dalla richiesta mutata o di regolare l'ordine dei filtri.
Tech stack
javaspring
Dominio
backendapi
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaSpring
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60