Add telemetry to protocol specification

[cryptocode]

nanocurrency/nano-node#2446

[wezrule]

telemetry_req
No payload

telemetry_ack
header.extensions contains the size of the telemetry data payload.

64 block_count
64 cemented_count
64 unchecked_count
64 account_count
64 bandwidth_cap
64 uptime
32 peer_count
8 protocol_version
256 genesis_block
8 major_version
8 minor_version
8 patch_version
8 pre_release_version
8 maker
64 timestamp

[inkeliz]

I think it should specify the type, not jus the size.

For comparation:

protocol/reference/protocol.ksy

Lines 288 to 296 in 047eb69

	vote_common:
	doc: Common data shared by block votes and vote-by-hash votes
	seq:
	- id: account
	size: 32
	- id: signature
	size: 64
	- id: sequence
	type: u8le

protocol/reference/protocol.ksy

Lines 352 to 380 in 047eb69

	msg_telemetry_ack:
	doc: Signed telemetry response
	seq:
	- id: signature
	size: 256
	doc: Signature
	- id: nodeid
	size: 256
	doc: Public node id
	- id: blockcount
	size: 64
	doc: Block count
	- id: cementedcount
	size: 64
	doc: Cemented block count
	- id: uncheckedcount
	size: 64
	doc: Unchecked block count
	- id: accountcount
	size: 64
	doc: Account count
	- id: bandwidthcap
	size: 64
	doc: Bandwidth limit, 0 indiciates unlimited
	- id: uptime
	size: 64
	doc: Length of time a peer has been running for (in seconds)
	- id: peercount
	size: 32

I think blockcount, for instance, is an uint64le, but it's not specified. I think it should be like the vote_common, with type: u8le.

[wezrule]

@inkeliz

Makes sense, thanks. https://github.com/nanocurrency/protocol/pull/12/files. In case it affects any testing, we have also modified the data so be consistent, it is all big endian now.

[zhyatt]

Fixed in #12