Merge pull request #702 from aantono/master

Fix for NPE when creating ProtobufHttpMessageConverter without ExtensionRegistryInitializer
master
Brian Clozel 10 years ago
commit 1de71511f0
  1. 2
      spring-web/src/main/java/org/springframework/http/converter/protobuf/ProtobufHttpMessageConverter.java
  2. 9
      spring-web/src/test/java/org/springframework/http/converter/protobuf/ProtobufHttpMessageConverterTests.java

@ -86,7 +86,7 @@ public class ProtobufHttpMessageConverter extends AbstractHttpMessageConverter<M
*/
public ProtobufHttpMessageConverter(ExtensionRegistryInitializer registryInitializer) {
super(PROTOBUF, MediaType.TEXT_PLAIN, MediaType.APPLICATION_XML, MediaType.APPLICATION_JSON);
if (this.extensionRegistry != null) {
if (registryInitializer != null) {
registryInitializer.initializeExtensionRegistry(this.extensionRegistry);
}
}

@ -58,6 +58,15 @@ public class ProtobufHttpMessageConverterTests {
verify(this.registryInitializer, times(1)).initializeExtensionRegistry(anyObject());
}
@Test
public void extensionRegistryNull() {
try {
new ProtobufHttpMessageConverter(null);
} catch (Exception e) {
fail("Unable to create ProtobufHttpMessageConverter with null extensionRegistry");
}
}
@Test
public void canRead() {
assertTrue(this.converter.canRead(Msg.class, null));

Loading…
Cancel
Save