fix: wrong operationHash doesnt match did

decentralized-identity · Dec 16, 2019 · 46d3a5c · 46d3a5c
1 parent 45d30c7
commit 46d3a5c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/packages/element-lib/src/sidetree-v2/func.js b/packages/element-lib/src/sidetree-v2/func.js
@@ -35,15 +35,14 @@ const payloadToHash = (payload) => {
 };
 
 const getDidUniqueSuffix = (operation) => {
-  const decodedPayload = decodeJson(operation.payload);
   const header = decodeJson(operation.protected);
   switch (header.operation) {
     case 'create':
-      return payloadToHash(decodedPayload);
+      return payloadToHash(operation.payload);
     case 'update':
     case 'recover':
     case 'delete':
-      return decodedPayload.didUniqueSuffix;
+      return decodeJson(operation.payload).didUniqueSuffix;
     default:
       throw Error(`Cannot extract didUniqueSuffixe from: ${operation}`);
   }

diff --git a/packages/element-lib/src/sidetree-v2/protocol/resolve.js b/packages/element-lib/src/sidetree-v2/protocol/resolve.js
@@ -1,5 +1,5 @@
 /* eslint-disable arrow-body-style */
-const { payloadToHash, verifyOperationSignature } = require('../func');
+const { verifyOperationSignature } = require('../func');
 const { isDidDocumentModelValid, isKeyValid } = require('../validation');
 
 const isSignatureValid = async (didDocument, operation) => {
@@ -32,7 +32,7 @@ const create = async (state, operation, lastValidOperation) => {
   await isSignatureValid(originalDidDocument, operation);
   return {
     ...operation.decodedOperationPayload,
-    id: `did:elem:${payloadToHash(operation.decodedOperationPayload)}`,
+    id: `did:elem:${operation.operationHash}`,
   };
 };