fankes/moshi
2
0
Fork 0
mirror of https://github.com/fankes/moshi.git synced 2025-10-19 16:09:21 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Revert "Don't allow toJson() serialization on polymorphic defau… (#1025)

Browse Source 
This reverts commit 6323a0b7c8.

This makes it difficult for a client to take a server-provided value,
modify it, and send it back.
This commit is contained in:
Jesse Wilson
2019-11-16 22:04:57 -05:00
committed by Zac Sweers
parent fde0ce0eb5
commit ca207b2e06
2 changed files with 0 additions and 21 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
adapters/src/main/java/com/squareup/moshi/adapters/PolymorphicJsonAdapterFactory.java
View File

@@ -275,9 +275,6 @@ public final class PolymorphicJsonAdapterFactory<T> implements JsonAdapter.Facto
} }
@Override public void toJson(JsonWriter writer, Object value) throws IOException { @Override public void toJson(JsonWriter writer, Object value) throws IOException {
if (defaultValueSet && value == defaultValue) {
throw new IllegalArgumentException("Cannot serialize default value instance: " + value);
}
Class<?> type = value.getClass(); Class<?> type = value.getClass();
int labelIndex = subtypes.indexOf(type); int labelIndex = subtypes.indexOf(type);
if (labelIndex == -1) { if (labelIndex == -1) {

18
adapters/src/test/java/com/squareup/moshi/adapters/PolymorphicJsonAdapterFactoryTest.java
View File

@@ -105,24 +105,6 @@ public final class PolymorphicJsonAdapterFactoryTest {
assertThat(message).isNull(); assertThat(message).isNull();
} }
@Test public void toJsonDefaultValue() {
Error fallbackError = new Error(Collections.<String, Object>emptyMap());
Moshi moshi = new Moshi.Builder()
.add(PolymorphicJsonAdapterFactory.of(Message.class, "type")
.withSubtype(Success.class, "success")
.withSubtype(Error.class, "error")
.withDefaultValue(fallbackError))
.build();
JsonAdapter<Message> adapter = moshi.adapter(Message.class);
try {
String json = adapter.toJson(fallbackError);
fail();
} catch (IllegalArgumentException e) {
assertThat(e).hasMessageContaining("Cannot serialize default value instance");
}
}
@Test public void unregisteredSubtype() { @Test public void unregisteredSubtype() {
Moshi moshi = new Moshi.Builder() Moshi moshi = new Moshi.Builder()
.add(PolymorphicJsonAdapterFactory.of(Message.class, "type") .add(PolymorphicJsonAdapterFactory.of(Message.class, "type")