Merge pull request #113 from Juneo-io/dev

v0.0.94

package.json

@@ -1,6 +1,6 @@
 {
   "name": "juneojs",
-  "version": "0.0.93",
+  "version": "0.0.94",
   "description": "Juneo JS Library",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

src/transaction/input.ts

@@ -1,5 +1,5 @@
 import { type Blockchain } from '../chain'
-import { JuneoBuffer, ParsingError, sha256, type Serializable, SignatureError, InputError } from '../utils'
+import { JuneoBuffer, ParsingError, type Serializable, SignatureError, InputError } from '../utils'
 import { type VMWallet } from '../wallet'
 import { type Utxo } from './output'
 import { type Signable } from './signature'
@@ -78,7 +78,7 @@ export class TransferableInput implements Serializable, Signable, Spendable {
       const address: Address = this.input.utxo.output.addresses[indices[i]]
       for (const wallet of wallets) {
         if (address.matches(wallet.getJuneoAddress())) {
-          signatures.push(new Signature(wallet.sign(sha256(bytes))))
+          signatures.push(new Signature(wallet.sign(bytes)))
           break
         }
       }

src/transaction/jevm/transaction.ts

@@ -1,4 +1,4 @@
-import { JuneoBuffer, sha256, type Serializable, SignatureError } from '../../utils'
+import { JuneoBuffer, type Serializable, SignatureError } from '../../utils'
 import { type VMWallet } from '../../wallet'
 import { type Spendable, TransferableInput } from '../input'
 import { TransferableOutput } from '../output'
@@ -74,7 +74,7 @@ export class EVMInput implements Serializable, Signable, Spendable {
     const address: Address = this.address
     for (const wallet of wallets) {
       if (address.matches(wallet.getAddress())) {
-        signatures.push(new Signature(wallet.sign(sha256(bytes))))
+        signatures.push(new Signature(wallet.sign(bytes)))
         break
       }
     }

src/transaction/platform/supernet.ts

@@ -1,4 +1,4 @@
-import { type Serializable, JuneoBuffer, SignatureError, sha256 } from '../../utils'
+import { type Serializable, JuneoBuffer, SignatureError } from '../../utils'
 import { type VMWallet } from '../../wallet'
 import { getSignersIndices } from '../builder'
 import { type Signable } from '../signature'
@@ -29,7 +29,7 @@ export class SupernetAuth implements Serializable, Signable {
       const address: Address = this.rewardsOwner.addresses[i]
       for (const wallet of wallets) {
         if (address.matches(wallet.getJuneoAddress())) {
-          signatures.push(new Signature(wallet.sign(sha256(bytes))))
+          signatures.push(new Signature(wallet.sign(bytes)))
           break
         }
       }

src/utils/bytes.ts

@@ -201,7 +201,9 @@ export class JuneoBuffer {
       }
     }
     if (fromEncoding.toLowerCase() === 'hex') {
-      return JuneoBuffer.fromBytes(Buffer.from(data, 'hex'))
+      return encoding.hasHexPrefix(data)
+        ? JuneoBuffer.fromBytes(Buffer.from(data.substring(2), 'hex'))
+        : JuneoBuffer.fromBytes(Buffer.from(data, 'hex'))
     } else if (fromEncoding.toLowerCase() === 'chex') {
       return encoding.decodeCHex(data)
     } else if (fromEncoding.toLowerCase() === 'cb58') {

src/utils/crypto.ts

@@ -13,16 +13,20 @@ export function sha256 (data: string | JuneoBuffer): JuneoBuffer {
   return JuneoBuffer.fromBytes(Buffer.from(nobleSha256(buffer.getBytes())))
 }
 
-export function recoverPubKey (signature: JuneoBuffer, hash: JuneoBuffer, recovery: number): string {
-  return JuneoBuffer.fromBytes(
-    Buffer.from(recoverPublicKey(hash.getBytes(), Signature.fromHex(signature.getBytes()), recovery, true))
-  )
-    .toHex()
-    .padStart(66, '0')
+export function recoverPubKey (signature: JuneoBuffer, message: JuneoBuffer, recovery: number): string {
+  const sig: Signature = parseSignature(signature)
+  const bytes: Buffer = Buffer.from(recoverPublicKey(nobleSha256(message.getBytes()), sig, recovery, true))
+  return JuneoBuffer.fromBytes(bytes).toHex().padStart(66, '0')
 }
 
-export function verifySignature (signature: JuneoBuffer, hash: JuneoBuffer, publicKey: string): boolean {
-  return verify(Signature.fromHex(signature.toHex()), hash.toHex(), publicKey)
+export function verifySignature (signature: JuneoBuffer, message: JuneoBuffer, publicKey: string): boolean {
+  return verify(parseSignature(signature), nobleSha256(message.getBytes()), publicKey)
+}
+
+function parseSignature (signature: JuneoBuffer): Signature {
+  return signature.length === 65
+    ? Signature.fromCompact(signature.getBytes().subarray(0, 64))
+    : Signature.fromCompact(signature.getBytes())
 }
 
 export class ECKeyPair {
@@ -36,10 +40,11 @@ export class ECKeyPair {
       .padStart(66, '0')
   }
 
-  sign (buffer: JuneoBuffer): JuneoBuffer {
-    const signature: Signature = Signature.fromHex(signSync(buffer.getBytes(), this.privateKey))
+  sign (message: JuneoBuffer): JuneoBuffer {
+    const signature: Signature = Signature.fromHex(signSync(nobleSha256(message.getBytes()), this.privateKey))
     // noble as of v1.7.1 does not provide recovery param so do it here
-    const v: number = recoverPubKey(JuneoBuffer.fromString(signature.toHex()), buffer, 0) === this.publicKey ? 0 : 1
+    const publicKey: string = recoverPubKey(JuneoBuffer.fromString(signature.toCompactHex()), message, 0)
+    const v: number = publicKey === this.publicKey ? 0 : 1
     return JuneoBuffer.fromString(`${signature.toCompactHex()}${v.toString(16).padStart(2, '0')}`)
   }
 }