Return value of util.serialize(dagNode.toJSON()) should produce equal value to dagNode.serialize()

[Gozala]

Current implementation of .toJSON() produces structure with fields: data, links and size. Which somewhat resembles DAGNode however passing that value into ipld.put stores block with CID QmdfTbBqBPQ7VNxZEYEj14VmRuZBkqFbiwReogJgS1zR1n corresponding to {} because call to util.serialize looks at the capital cased fields instead

js-ipld-dag-pb/src/serialize.js

Lines 40 to 50 in 1012cad

	const serializeDAGNode = (node) => {
	const data = node.Data
	const links = node.Links || []
	const serialized = proto.PBNode.encode(toProtoBuf({
	Data: data,
	Links: links
	}))
	return serialized
	}

Which I find really unfortunate, as I was hoping to toJSON() for posting nodes across the threads.

[Gozala]

For my specific use case it would be even better if Data and Links getters were just turned into regular properties. That would make it unnecessary to call toJSON at least for my use case.

[vmx]

toJSON() is mostly there for test purpose. It shouldn't really be used. I would've long removed it (as in other codec implementations) if it wouldn't break so much.