Throw errors instead of returning null when serialization/ deserialization fails

[ben-xD]

In Java, if serialization/ deserialization fails, instead of propagating the error, it is logged and ignored, and null is returned instead. For example:

ably-flutter/android/src/main/java/io/ably/flutter/plugin/AblyMessageCodec.java

Lines 348 to 357 in 21c49b9

	private ChannelOptions decodeRestChannelOptions(Map<String, Object> jsonMap) {
	if (jsonMap == null) return null;
	final Object cipher = jsonMap.get(PlatformConstants.TxRealtimeChannelOptions.cipher);
	try {
	return ChannelOptions.withCipherKey((String) cipher);
	} catch (AblyException ae) {
	System.out.println("Exception while decoding RestChannelOptions : " + ae);
	return null;
	}
	}

It would be nicer during development to have an exception be thrown when serialization fails. We should consider throwing an exception when serialization/ deserialization fails.

┆Issue is synchronized with this Jira Story by Unito

[QuintinWillison]

See this comment.

[ikurek]

Working link to mentioned comment: #140 (comment)