spring-cloud/spring-cloud-gateway

Modify Request Body for a GET request

Offen

#1.147 geöffnet am 03.07.2019

Auf GitHub ansehenContributor Guide
 (7 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Java (3.204 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (4.284 Sterne)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

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?

Contributor Guide

Research-Richtung
Untersuchen Sie die aktuelle Implementierung von `ModifyRequestBodyGatewayFilterFactory` im spring cloud gateway Repository. Der vom Benutzer vorgeschlagene Fix besteht darin, `.defaultIfEmpty("")` in der `filter` Methode hinzuzufügen, um das Umschreiben des Bodys zu ermöglichen, wenn der ursprüngliche Anfrage Body leer ist (typisch für GET Anfragen). Implementieren Sie diese Änderung und stellen Sie sicher, dass sie das bestehende Verhalten für POST Anfragen mit einem Body nicht beeinträchtigt. Schreiben Sie Unit Tests, die beide Szenarien abdecken, und verifizieren Sie, dass der Filter mit der standardmäßigen `ModifyRequestBodyGatewayFilterFactory` funktioniert. Weitere Informationen zum erwarteten Verhalten finden Sie im verlinkten Issue.
Tech Stack
javaspring
Domain
backendapicloud
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaGitSpring
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70