Configure String codecs in RSocket strategies

Closes gh-16924

Author: bclozel

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/rsocket/RSocketStrategiesAutoConfiguration.java

@@ -33,6 +33,8 @@
 import org.springframework.context.annotation.Configuration;
 import org.springframework.core.ReactiveAdapterRegistry;
 import org.springframework.core.annotation.Order;
+import org.springframework.core.codec.CharSequenceEncoder;
+import org.springframework.core.codec.StringDecoder;
 import org.springframework.core.io.buffer.NettyDataBufferFactory;
 import org.springframework.http.MediaType;
 import org.springframework.http.codec.cbor.Jackson2CborDecoder;
@@ -62,6 +64,8 @@ public RSocketStrategies rSocketStrategies(
 		builder.reactiveAdapterStrategy(ReactiveAdapterRegistry.getSharedInstance());
 		customizers.orderedStream()
 				.forEach((customizer) -> customizer.customize(builder));
+		builder.decoder(StringDecoder.textPlainOnly());
+		builder.encoder(CharSequenceEncoder.textPlainOnly());
 		builder.dataBufferFactory(
 				new NettyDataBufferFactory(PooledByteBufAllocator.DEFAULT));
 		return builder.build();

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/rsocket/RSocketStrategiesAutoConfigurationTests.java

@@ -24,6 +24,8 @@
 import org.springframework.boot.test.context.runner.ApplicationContextRunner;
 import org.springframework.context.annotation.Bean;
 import org.springframework.context.annotation.Configuration;
+import org.springframework.core.codec.ByteArrayDecoder;
+import org.springframework.core.codec.ByteArrayEncoder;
 import org.springframework.core.codec.CharSequenceEncoder;
 import org.springframework.core.codec.StringDecoder;
 import org.springframework.http.codec.cbor.Jackson2CborDecoder;
@@ -51,16 +53,19 @@ public void shouldCreateDefaultBeans() {
 		this.contextRunner.run((context) -> {
 			assertThat(context).getBeans(RSocketStrategies.class).hasSize(1);
 			RSocketStrategies strategies = context.getBean(RSocketStrategies.class);
-			assertThat(strategies.decoders()).hasSize(2);
+			assertThat(strategies.decoders()).hasSize(3);
 			assertThat(strategies.decoders().get(0))
 					.isInstanceOf(Jackson2CborDecoder.class);
 			assertThat(strategies.decoders().get(1))
 					.isInstanceOf(Jackson2JsonDecoder.class);
-			assertThat(strategies.encoders()).hasSize(2);
+			assertThat(strategies.decoders().get(2)).isInstanceOf(StringDecoder.class);
+			assertThat(strategies.encoders()).hasSize(3);
 			assertThat(strategies.encoders().get(0))
 					.isInstanceOf(Jackson2CborEncoder.class);
 			assertThat(strategies.encoders().get(1))
 					.isInstanceOf(Jackson2JsonEncoder.class);
+			assertThat(strategies.encoders().get(2))
+					.isInstanceOf(CharSequenceEncoder.class);
 		});
 	}
 
@@ -80,10 +85,10 @@ public void shouldUseStrategiesCustomizer() {
 					assertThat(context).getBeans(RSocketStrategies.class).hasSize(1);
 					RSocketStrategies strategies = context
 							.getBean(RSocketStrategies.class);
-					assertThat(strategies.decoders()).hasSize(3)
-							.hasAtLeastOneElementOfType(StringDecoder.class);
-					assertThat(strategies.encoders()).hasSize(3)
-							.hasAtLeastOneElementOfType(CharSequenceEncoder.class);
+					assertThat(strategies.decoders()).hasSize(4)
+							.hasAtLeastOneElementOfType(ByteArrayDecoder.class);
+					assertThat(strategies.encoders()).hasSize(4)
+							.hasAtLeastOneElementOfType(ByteArrayEncoder.class);
 				});
 	}
 
@@ -119,8 +124,8 @@ static class StrategiesCustomizer {
 
 		@Bean
 		public RSocketStrategiesCustomizer myCustomizer() {
-			return (strategies) -> strategies.encoder(CharSequenceEncoder.textPlainOnly())
-					.decoder(StringDecoder.textPlainOnly());
+			return (strategies) -> strategies.encoder(new ByteArrayEncoder())
+					.decoder(new ByteArrayDecoder());
 		}
 
 	}