search

veres-one / veres-one-validator

A ledger validator that accepts either signatures or proof of work
Other
3 stars 0 forks source link

Add hard limits for operation and event sizes #3

Closed dlongley closed 4 years ago

mattcollier commented 4 years ago

We implemented size restrictions on operation sizes in Continuity here: https://github.com/digitalbazaar/bedrock-ledger-consensus-continuity/blob/master/lib/events.js#L771-L781

Is there anything here to be done related to this?

dlongley commented 4 years ago

@mattcollier,

Is there anything here to be done related to this?

Is there a hard limit on the number of operations per event as well? If not, we need to close that loop as well.

mattcollier commented 4 years ago

Yes, there is a hard coded limit of 250 operations per event: https://github.com/digitalbazaar/bedrock-ledger-consensus-continuity/blob/master/lib/continuityConstants.js#L11-L24

dlongley commented 4 years ago

@mattcollier --- then we're done here.

dlongley commented 4 years ago

Thanks!