fix: MosaicMetadataTransaction, NamespaceMetadataTransaction value string type changed to byte array

Baha · Baha · commit 523ba512dc2a · 2022-02-13T21:07:34.000Z
diff --git a/e2e/infrastructure/MetadataHttp.spec.ts b/e2e/infrastructure/MetadataHttp.spec.ts
@@ -141,7 +141,7 @@ describe('MetadataHttp', () => {
                 UInt64.fromUint(6),
                 mosaicId,
                 22,
-                `Test mosaic meta value`,
+                Convert.utf8ToUint8(`Test mosaic meta value`),
                 networkType,
                 helper.maxFee,
             );
@@ -166,7 +166,7 @@ describe('MetadataHttp', () => {
                 UInt64.fromUint(6),
                 namespaceId,
                 25,
-                `Test namespace meta value`,
+                Convert.utf8ToUint8(`Test namespace meta value`),
                 networkType,
                 helper.maxFee,
             );
diff --git a/e2e/infrastructure/TransactionHttp.spec.ts b/e2e/infrastructure/TransactionHttp.spec.ts
@@ -229,7 +229,7 @@ describe('TransactionHttp', () => {
                 UInt64.fromUint(5),
                 mosaicId,
                 10,
-                Convert.uint8ToUtf8(new Uint8Array(10)),
+                new Uint8Array(10),
                 networkType,
                 helper.maxFee,
             );
@@ -323,7 +323,7 @@ describe('TransactionHttp', () => {
                 UInt64.fromUint(5),
                 addressAlias,
                 10,
-                Convert.uint8ToUtf8(new Uint8Array(10)),
+                new Uint8Array(10),
                 networkType,
                 helper.maxFee,
             );
diff --git a/e2e/infrastructure/UnresolvedMapping.spec.ts b/e2e/infrastructure/UnresolvedMapping.spec.ts
@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 import { expect } from 'chai';
-import { Convert } from '../../src/core/format';
 import { Account } from '../../src/model/account';
 import { PlainMessage } from '../../src/model/message/PlainMessage';
 import { MosaicFlags, MosaicId, MosaicNonce } from '../../src/model/mosaic';
@@ -165,7 +164,7 @@ describe('Unresolved Mapping', () => {
                 UInt64.fromUint(5),
                 namespaceIdMosaic,
                 10,
-                Convert.uint8ToUtf8(new Uint8Array(10)),
+                new Uint8Array(10),
                 networkType,
                 helper.maxFee,
             );
diff --git a/e2e/service/MetadataTransactionService.spec.ts b/e2e/service/MetadataTransactionService.spec.ts
@@ -95,7 +95,7 @@ describe('MetadataTransactionService', () => {
                 key,
                 mosaicId,
                 newValue.length,
-                newValue,
+                Convert.utf8ToUint8(newValue),
                 networkType,
                 helper.maxFee,
             );
@@ -121,7 +121,7 @@ describe('MetadataTransactionService', () => {
                 key,
                 namespaceId,
                 newValue.length,
-                newValue,
+                Convert.utf8ToUint8(newValue),
                 networkType,
             );
 
diff --git a/e2e/service/TransactionService.spec.ts b/e2e/service/TransactionService.spec.ts
@@ -16,7 +16,6 @@
 
 import { assert, expect } from 'chai';
 import { firstValueFrom } from 'rxjs';
-import { Convert } from '../../src/core/format';
 import { TransactionRepository } from '../../src/infrastructure/TransactionRepository';
 import { Account, Address } from '../../src/model/account';
 import { PlainMessage } from '../../src/model/message/PlainMessage';
@@ -136,7 +135,7 @@ describe('TransactionService', () => {
             UInt64.fromUint(5),
             mosaicAlias,
             10,
-            Convert.uint8ToUtf8(new Uint8Array(10)),
+            new Uint8Array(10),
             networkType,
             helper.maxFee,
         );
diff --git a/src/core/format/Convert.ts b/src/core/format/Convert.ts
@@ -26,7 +26,7 @@ export class Convert {
     public static toByte = (char1: string, char2: string): number => {
         const byte = utilities.tryParseByte(char1, char2);
         if (undefined === byte) {
-            throw Error(`unrecognized hex char`);
+            throw Error(`unrecognized hex char, char1:${char1}, char2:${char2}`);
         }
         return byte;
     };
@@ -148,33 +148,6 @@ export class Convert {
         return input & 0xff;
     };
 
-    /**
-     * Converts a raw javascript string into a string of single byte characters using utf8 encoding.
-     * This makes it easier to perform other encoding operations on the string.
-     * @param {string} input - A raw string
-     * @return {string} - UTF-8 string
-     */
-    public static rstr2utf8 = (input: string): string => {
-        let output = '';
-
-        for (let n = 0; n < input.length; n++) {
-            const c = input.charCodeAt(n);
-
-            if (128 > c) {
-                output += String.fromCharCode(c);
-            } else if (127 < c && 2048 > c) {
-                output += String.fromCharCode((c >> 6) | 192);
-                output += String.fromCharCode((c & 63) | 128);
-            } else {
-                output += String.fromCharCode((c >> 12) | 224);
-                output += String.fromCharCode(((c >> 6) & 63) | 128);
-                output += String.fromCharCode((c & 63) | 128);
-            }
-        }
-
-        return output;
-    };
-
     /**
      * Convert UTF-8 to hex
      * @param {string} input - An UTF-8 string
diff --git a/src/infrastructure/transaction/CreateTransactionFromDTO.ts b/src/infrastructure/transaction/CreateTransactionFromDTO.ts
@@ -13,7 +13,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { Convert as convert } from '../../core/format';
 import { UnresolvedMapping } from '../../core/utils';
 import { MessageFactory, MosaicSupplyRevocationTransaction, UInt64 } from '../../model';
 import { Address, PublicAccount } from '../../model/account';
@@ -393,7 +392,7 @@ const CreateStandaloneTransactionFromDTO = (transactionDTO, transactionInfo): Tr
                 extractRecipient(transactionDTO.targetAddress),
                 UInt64.fromHex(transactionDTO.scopedMetadataKey),
                 transactionDTO.valueSizeDelta,
-                convert.utf8ToUint8(transactionDTO.value),
+                transactionDTO.value,
                 signature,
                 signer,
                 transactionInfo,
@@ -408,7 +407,7 @@ const CreateStandaloneTransactionFromDTO = (transactionDTO, transactionInfo): Tr
                 UInt64.fromHex(transactionDTO.scopedMetadataKey),
                 UnresolvedMapping.toUnresolvedMosaic(transactionDTO.targetMosaicId),
                 transactionDTO.valueSizeDelta,
-                convert.decodeHex(transactionDTO.value),
+                transactionDTO.value,
                 signature,
                 signer,
                 transactionInfo,
@@ -423,7 +422,7 @@ const CreateStandaloneTransactionFromDTO = (transactionDTO, transactionInfo): Tr
                 UInt64.fromHex(transactionDTO.scopedMetadataKey),
                 NamespaceId.createFromEncoded(transactionDTO.targetNamespaceId),
                 transactionDTO.valueSizeDelta,
-                convert.decodeHex(transactionDTO.value),
+                transactionDTO.value,
                 signature,
                 signer,
                 transactionInfo,
diff --git a/src/infrastructure/transaction/SerializeTransactionToJSON.ts b/src/infrastructure/transaction/SerializeTransactionToJSON.ts
@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-import { Convert } from '../../core/format';
 import {
     AccountAddressRestrictionTransaction,
     AccountKeyLinkTransaction,
@@ -256,7 +255,7 @@ export const SerializeTransactionToJSON = (transaction: Transaction): any => {
             valueSizeDelta: mosaicMetadataTx.valueSizeDelta,
             targetMosaicId: mosaicMetadataTx.targetMosaicId.id.toHex(),
             valueSize: mosaicMetadataTx.value.length,
-            value: Convert.utf8ToHex(mosaicMetadataTx.value),
+            value: mosaicMetadataTx.value,
         };
     } else if (transaction.type === TransactionType.NAMESPACE_METADATA) {
         const namespaceMetaTx = transaction as NamespaceMetadataTransaction;
@@ -266,7 +265,7 @@ export const SerializeTransactionToJSON = (transaction: Transaction): any => {
             valueSizeDelta: namespaceMetaTx.valueSizeDelta,
             targetNamespaceId: namespaceMetaTx.targetNamespaceId.id.toHex(),
             valueSize: namespaceMetaTx.value.length,
-            value: Convert.utf8ToHex(namespaceMetaTx.value),
+            value: namespaceMetaTx.value,
         };
     } else if (transaction.type === TransactionType.VRF_KEY_LINK) {
         const vrfKeyLinkTx = transaction as VrfKeyLinkTransaction;
diff --git a/src/model/transaction/AccountMetadataTransaction.ts b/src/model/transaction/AccountMetadataTransaction.ts
@@ -46,8 +46,7 @@ export class AccountMetadataTransaction extends Transaction {
      * @param targetAddress - target account address.
      * @param scopedMetadataKey - Metadata key scoped to source, target and type.
      * @param valueSizeDelta - Change in value size in bytes.
-     * @param value - String value with UTF-8 encoding
-     *                Difference between the previous value and new value.
+     * @param value - Difference between the previous value and new value.
      *                You can calculate value as xor(previous-value, new-value).
      *                If there is no previous value, use directly the new value.
      * @param maxFee - (Optional) Max fee defined by the sender
diff --git a/src/model/transaction/MosaicMetadataTransaction.ts b/src/model/transaction/MosaicMetadataTransaction.ts
@@ -51,8 +51,7 @@ export class MosaicMetadataTransaction extends Transaction {
      * @param scopedMetadataKey - Metadata key scoped to source, target and type.
      * @param targetMosaicId - Target unresolved mosaic identifier.
      * @param valueSizeDelta - Change in value size in bytes.
-     * @param value - String value with UTF-8 encoding
-     *                Difference between the previous value and new value.
+     * @param value - Difference between the previous value and new value.
      *                You can calculate value as xor(previous-value, new-value).
      *                If there is no previous value, use directly the new value.
      * @param maxFee - (Optional) Max fee defined by the sender
@@ -66,7 +65,7 @@ export class MosaicMetadataTransaction extends Transaction {
         scopedMetadataKey: UInt64,
         targetMosaicId: UnresolvedMosaicId,
         valueSizeDelta: number,
-        value: string,
+        value: Uint8Array,
         networkType: NetworkType,
         maxFee: UInt64 = new UInt64([0, 0]),
         signature?: string,
@@ -123,10 +122,10 @@ export class MosaicMetadataTransaction extends Transaction {
          */
         public readonly valueSizeDelta: number,
         /**
-         * String value with UTF-8 encoding.
+         * xor of previous and the new value
          * Difference between the previous value and new value.
          */
-        public readonly value: string,
+        public readonly value: Uint8Array,
         signature?: string,
         signer?: PublicAccount,
         transactionInfo?: TransactionInfo,
@@ -155,7 +154,7 @@ export class MosaicMetadataTransaction extends Transaction {
             new UInt64(builder.getScopedMetadataKey()),
             UnresolvedMapping.toUnresolvedMosaic(new UInt64(builder.getTargetMosaicId().unresolvedMosaicId).toHex()),
             builder.getValueSizeDelta(),
-            Convert.uint8ToUtf8(builder.getValue()),
+            builder.getValue(),
             networkType,
             isEmbedded ? new UInt64([0, 0]) : new UInt64((builder as MosaicMetadataTransactionBuilder).fee.amount),
             signature,
@@ -181,7 +180,7 @@ export class MosaicMetadataTransaction extends Transaction {
             this.scopedMetadataKey.toDTO(),
             new UnresolvedMosaicIdDto(this.targetMosaicId.id.toDTO()),
             this.valueSizeDelta,
-            Convert.utf8ToUint8(this.value),
+            this.value,
         );
         return transactionBuilder;
     }
@@ -200,7 +199,7 @@ export class MosaicMetadataTransaction extends Transaction {
             this.scopedMetadataKey.toDTO(),
             new UnresolvedMosaicIdDto(this.targetMosaicId.id.toDTO()),
             this.valueSizeDelta,
-            Convert.utf8ToUint8(this.value),
+            this.value,
         );
     }
 
diff --git a/src/model/transaction/NamespaceMetadataTransaction.ts b/src/model/transaction/NamespaceMetadataTransaction.ts
@@ -48,8 +48,7 @@ export class NamespaceMetadataTransaction extends Transaction {
      * @param scopedMetadataKey - Metadata key scoped to source, target and type.
      * @param targetNamespaceId - Target namespace identifier.
      * @param valueSizeDelta - Change in value size in bytes.
-     * @param value - String value with UTF-8 encoding
-     *                Difference between the previous value and new value.
+     * @param value - Difference between the previous value and new value.
      *                You can calculate value as xor(previous-value, new-value).
      *                If there is no previous value, use directly the new value.
      * @param maxFee - (Optional) Max fee defined by the sender
@@ -63,7 +62,7 @@ export class NamespaceMetadataTransaction extends Transaction {
         scopedMetadataKey: UInt64,
         targetNamespaceId: NamespaceId,
         valueSizeDelta: number,
-        value: string,
+        value: Uint8Array,
         networkType: NetworkType,
         maxFee: UInt64 = new UInt64([0, 0]),
         signature?: string,
@@ -120,10 +119,10 @@ export class NamespaceMetadataTransaction extends Transaction {
          */
         public readonly valueSizeDelta: number,
         /**
-         * String value with UTF-8 encoding.
+         * xor of previous and the new value
          * Difference between the previous value and new value.
          */
-        public readonly value: string,
+        public readonly value: Uint8Array,
         signature?: string,
         signer?: PublicAccount,
         transactionInfo?: TransactionInfo,
@@ -152,7 +151,7 @@ export class NamespaceMetadataTransaction extends Transaction {
             new UInt64(builder.getScopedMetadataKey()),
             new NamespaceId(builder.getTargetNamespaceId().namespaceId),
             builder.getValueSizeDelta(),
-            Convert.uint8ToUtf8(builder.getValue()),
+            builder.getValue(),
             networkType,
             isEmbedded ? new UInt64([0, 0]) : new UInt64((builder as NamespaceMetadataTransactionBuilder).fee.amount),
             signature,
@@ -178,7 +177,7 @@ export class NamespaceMetadataTransaction extends Transaction {
             this.scopedMetadataKey.toDTO(),
             this.targetNamespaceId.toBuilder(),
             this.valueSizeDelta,
-            Convert.utf8ToUint8(this.value),
+            this.value,
         );
         return transactionBuilder;
     }
@@ -197,7 +196,7 @@ export class NamespaceMetadataTransaction extends Transaction {
             this.scopedMetadataKey.toDTO(),
             this.targetNamespaceId.toBuilder(),
             this.valueSizeDelta,
-            Convert.utf8ToUint8(this.value),
+            this.value,
         );
     }
 
diff --git a/src/service/MetadataTransactionService.ts b/src/service/MetadataTransactionService.ts
@@ -128,13 +128,15 @@ export class MetadataTransactionService {
                         const metadata = metadatas.data[0];
                         const currentValueByte = Convert.utf8ToUint8(metadata.metadataEntry.value);
                         const newValueBytes = Convert.utf8ToUint8(value);
+                        const xoredBytes = Convert.hexToUint8(Convert.xor(currentValueByte, newValueBytes));
+
                         return MosaicMetadataTransaction.create(
                             deadline,
                             targetAddress,
                             key,
                             mosaicId,
                             newValueBytes.length - currentValueByte.length,
-                            Convert.decodeHex(Convert.xor(currentValueByte, newValueBytes)),
+                            xoredBytes,
                             networkType,
                             maxFee,
                         );
@@ -146,7 +148,7 @@ export class MetadataTransactionService {
                         key,
                         mosaicId,
                         newValueBytes.length,
-                        value,
+                        Convert.utf8ToUint8(value),
                         networkType,
                         maxFee,
                     );
@@ -192,13 +194,14 @@ export class MetadataTransactionService {
                         const metadata = metadatas.data[0];
                         const currentValueByte = Convert.utf8ToUint8(metadata.metadataEntry.value);
                         const newValueBytes = Convert.utf8ToUint8(value);
+                        const xoredBytes = Convert.hexToUint8(Convert.xor(currentValueByte, newValueBytes));
                         return NamespaceMetadataTransaction.create(
                             deadline,
                             targetAddress,
                             key,
                             namespaceId,
                             newValueBytes.length - currentValueByte.length,
-                            Convert.decodeHex(Convert.xor(currentValueByte, newValueBytes)),
+                            xoredBytes,
                             networkType,
                             maxFee,
                         );
@@ -210,7 +213,7 @@ export class MetadataTransactionService {
                         key,
                         namespaceId,
                         newValueBytes.length,
-                        value,
+                        Convert.utf8ToUint8(value),
                         networkType,
                         maxFee,
                     );
diff --git a/test/core/utils/TransactionMapping.spec.ts b/test/core/utils/TransactionMapping.spec.ts
diff --git a/test/core/utils/TransactionMappingWithSignatures.spec.ts b/test/core/utils/TransactionMappingWithSignatures.spec.ts
diff --git a/test/model/transaction/MosaicMetadataTransaction.spec.ts b/test/model/transaction/MosaicMetadataTransaction.spec.ts
diff --git a/test/model/transaction/NamespaceMetadataTransaction.spec.ts b/test/model/transaction/NamespaceMetadataTransaction.spec.ts
diff --git a/test/service/MetadataTransactionservice.spec.ts b/test/service/MetadataTransactionservice.spec.ts
