From 65246f8cfdcda92288e96209edbbdad3a4321a80 Mon Sep 17 00:00:00 2001 From: Rossen Stoyanchev Date: Wed, 1 Jun 2016 10:17:12 -0400 Subject: [PATCH] Remove FIXME on retain in ReactorServerHttpRequest The converters release so this is the right thing to do, not a temporary fix. --- .../http/server/reactive/ReactorServerHttpRequest.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/spring-web-reactive/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java b/spring-web-reactive/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java index bdd587c0ca4..8defae9cb64 100644 --- a/spring-web-reactive/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java +++ b/spring-web-reactive/src/main/java/org/springframework/http/server/reactive/ReactorServerHttpRequest.java @@ -89,9 +89,7 @@ public class ReactorServerHttpRequest extends AbstractServerHttpRequest { @Override public Flux getBody() { - return this.channel.receive() - .retain() //FIXME Rogue reference holding - .map(dataBufferFactory::wrap); + return this.channel.receive().retain().map(this.dataBufferFactory::wrap); } }