spring-cloud/spring-cloud-gateway

Modify Request Body for a GET request

Open

#1147 aperta il 3 lug 2019

Vedi su GitHub
 (7 commenti) (0 reazioni) (0 assegnatari)Java (3204 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (4284 star)
Metriche merge PR
 (Merge medio 1g 21h) (18 PR mergiate in 30 g)

Descrizione

We currently have custom code for modifying request body which works but I want to move to the standard ModifyRequestBodyGatewayFilterFactory.

We have a scenario where Spring Cloud Gateway received a GET request and we need call an internal GraphQL api with a POST and a request body.

ModifyRequestBodyGatewayFilterFactory works great if there is a body present in the original request but doesn't apply the rewriteFunction when there is no body on GET requests.

I am using v2.1.2.RELEASE and I have hacked together a change to ModifyRequestBodyGatewayFilterFactory which works for me:

Mono<?> modifiedBody = serverRequest.bodyToMono(inClass)
        .defaultIfEmpty("") //added this line
        .flatMap(o -> config.rewriteFunction.apply(exchange, o));

I appreciate that the ModifyRequestBodyGatewayFilterFactory is beta, but would it be possible to support adding a body for GET requests in the future?

Guida contributor

Direzione di ricerca
Esamina l'implementazione corrente di `ModifyRequestBodyGatewayFilterFactory` nel repository spring cloud gateway. La correzione proposta dall'utente consiste nell'aggiungere `.defaultIfEmpty("")` nel metodo `filter` per consentire la riscrittura del corpo quando il corpo della richiesta originale è vuoto (tipico per le richieste GET). Implementa questa modifica e assicurati che non interrompa il comportamento esistente per le richieste POST con un corpo. Scrivi test unitari che coprano entrambi gli scenari e verifica che il filtro funzioni con lo standard `ModifyRequestBodyGatewayFilterFactory`. Fai riferimento all'issue collegata per ulteriore contesto sul comportamento previsto.
Tech stack
javaspring
Dominio
backendapicloud
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaGitSpring
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70