disabled arweave signatures, we are moving to EVM signatures only

Author: hatskier

__tests__/fluent-interface.test.ts

@@ -1,6 +1,6 @@
 import redstone from "../src/index";
 
-const MAX_TIME_DIFF = 120000; // 120s
+const MAX_TIME_DIFF = 150000; // 150s
 
 describe("Fluent interface tests ", () => {
 

__tests__/get-all-prices.test.ts

@@ -1,6 +1,6 @@
 import redstone from "../src/index";
 
-const MAX_TIME_DIFF = 120000; // 120s
+const MAX_TIME_DIFF = 150000; // 150s
 const MAX_TIME_DIFF_ARWEAVE = 7200 * 1000; // 2 hours
 
 const shouldNotHaveTechProps = (price: any) => {
@@ -37,15 +37,4 @@ describe("Test getAllPrices method", () => {
     }
   });
 
-  test("Should get all prices from arweave", async () => {
-    const noCacheApi = new redstone.Api({ useCache: false });
-    const prices = await noCacheApi.getAllPrices();
-    expect(Object.keys(prices).length).toBeGreaterThan(3);
-    expect(Date.now() - prices["AR"].timestamp).toBeLessThan(MAX_TIME_DIFF_ARWEAVE);
-    expect(Date.now() - prices["BTC"].timestamp).toBeLessThan(MAX_TIME_DIFF_ARWEAVE);
-    for (const price of Object.values(prices)) {
-      shouldNotHaveTechProps(price);
-    }
-  });
-
 });

src/signature-verifier.ts

@@ -11,32 +11,35 @@ export default class SignatureVerifier {
   }
 
   public async assertValidSignature(
-    price: PriceDataWithSignature,
+    _price: PriceDataWithSignature,
   ): Promise<void> {
-    const signedData = this.getPriceSignedData(price);
-    const publicKey = price.providerPublicKey;
-
-    const validSignature = await this.arweaveProxy.verifySignature({
-      signedData,
-      signature: price.signature,
-      signerPublicKey: publicKey,
-    });
-
-    const addressFromPublicKey = await this.arweaveProxy.arweaveClient.wallets.ownerToAddress(
-      publicKey,
-    );
-
-    if (!validSignature) {
-      throw new Error("Signature verification failed for price: " + signedData);
-    }
-
-    if (addressFromPublicKey !== price.provider) {
-      throw new Error(
-        `Provider address doesn't match the public key.` +
-          ` Address: ${price.provider}.` +
-          ` Public key: ${publicKey}.`,
-      );
-    }
+    // We've decided to disable signing with Arweave signature
+    // and use only EVM compatible signatures instead
+
+    // const signedData = this.getPriceSignedData(price);
+    // const publicKey = price.providerPublicKey;
+
+    // const validSignature = await this.arweaveProxy.verifySignature({
+    //   signedData,
+    //   signature: price.signature,
+    //   signerPublicKey: publicKey,
+    // });
+
+    // const addressFromPublicKey = await this.arweaveProxy.arweaveClient.wallets.ownerToAddress(
+    //   publicKey,
+    // );
+
+    // if (!validSignature) {
+    //   throw new Error("Signature verification failed for price: " + signedData);
+    // }
+
+    // if (addressFromPublicKey !== price.provider) {
+    //   throw new Error(
+    //     `Provider address doesn't match the public key.` +
+    //       ` Address: ${price.provider}.` +
+    //       ` Public key: ${publicKey}.`,
+    //   );
+    // }
   }
 
   private getPriceSignedData(price: PriceDataWithSignature) {