feat(ref-imp): #781 - make jcs long form do size check on delta

lib/core/versions/latest/Delta.ts

@@ -1,6 +1,7 @@
 import ErrorCode from './ErrorCode';
 import ProtocolParameters from './ProtocolParameters';
 import SidetreeError from '../../../common/SidetreeError';
+import JsonCanonicalizer from './util/JsonCanonicalizer';
 
 /**
  * Class containing reusable operation delta functionalities.
@@ -9,6 +10,7 @@ export default class Delta {
 
   /**
    * Validates size of the encoded delta string.
+   * TODO: SIP 2 #781 delete this and use validateDeltaSize only
    * @throws `SidetreeError` if fails validation.
    */
   public static validateEncodedDeltaSize (encodedDelta: string) {
@@ -19,4 +21,16 @@ export default class Delta {
       throw new SidetreeError(ErrorCode.DeltaExceedsMaximumSize, errorMessage);
     }
   }
+
+  /**
+   * Validates size of the delta object
+   */
+  public static validateDeltaSize (delta: object) {
+    const size = Buffer.byteLength(JsonCanonicalizer.canonicalizeAsBuffer(delta));
+    if (size > ProtocolParameters.maxDeltaSizeInBytes) {
+      const errorMessage = `${size} bytes of 'delta' exceeded limit of ${ProtocolParameters.maxDeltaSizeInBytes} bytes.`;
+      console.info(errorMessage);
+      throw new SidetreeError(ErrorCode.DeltaExceedsMaximumSize, errorMessage);
+    }
+  }
 }

lib/core/versions/latest/Did.ts

@@ -178,6 +178,7 @@ export default class Did {
     }
 
     Did.validateInitialState(initialStateEncodedJcs, initialStateObject);
+    Delta.validateDeltaSize(initialStateObject.delta);
 
     const createOperationRequest = {
       type: OperationType.Create,

lib/core/versions/latest/protocol-parameters.json

@@ -6,7 +6,7 @@
   "maxNumberOfOperationsPerTransactionTime": 600000,
   "maxNumberOfTransactionsPerTransactionTime": 300,
   "maxOperationsPerBatch": 10000,
-  "maxDeltaSizeInBytes": 1000,
+  "maxDeltaSizeInBytes": 2000,
   "maxNumberOfOperationsForNoValueTimeLock": 100,
   "maxWriterLockIdInBytes": 200,
   "normalizedFeeToPerOperationFeeMultiplier": 0.01,

tests/core/Did.spec.ts

@@ -3,6 +3,7 @@ import Did from '../../lib/core/versions/latest/Did';
 import Encoder from '../../lib/core/versions/latest/Encoder';
 import ErrorCode from '../../lib/core/versions/latest/ErrorCode';
 import JasmineSidetreeErrorValidator from '../JasmineSidetreeErrorValidator';
+import JsonCanonicalizer from '../../lib/core/versions/latest/util/JsonCanonicalizer';
 import OperationGenerator from '../generators/OperationGenerator';
 import SidetreeError from '../../lib/common/SidetreeError';
 
@@ -27,6 +28,19 @@ describe('DID', async () => {
         expect(e).toEqual(new SidetreeError(ErrorCode.DidInitialStateJcsIsNotJcs, 'Initial state object and JCS string mismatch.'));
       }
     });
+
+    it('should throw sidetree error if delta exceeds size limit', () => {
+      const largeData = crypto.randomBytes(2000).toString('hex');// Intentionally exceeding max size.
+      const largeDelta = { data: largeData };
+      const testInitialState = Encoder.encode(JsonCanonicalizer.canonicalizeAsBuffer({ suffix_data: 'some data', delta: largeDelta }));
+
+      try {
+        Did['constructCreateOperationFromEncodedJcs'](testInitialState);
+        fail('expect to throw sidetree error but did not');
+      } catch (e) {
+        expect(e.code).toEqual(ErrorCode.DeltaExceedsMaximumSize);
+      }
+    })
   });
 
   describe('create()', async () => {