[WIP] LitEncryption Module #130

Juliano1612 · 2023-04-25T17:49:17Z

Description

This adds support for the LitEncryption module.

Type

New feature (non-breaking change which adds functionality)

Diligence Checklist

This change requires a documentation update
I have included unit tests
I have performed a self-review of my code
I have commented my code, particularly in hard-to-understand areas
I have made corresponding changes to the documentation
My changes generate no new warnings
Any dependent changes have been merged and published in downstream modules

* ssx design pattern * updated tests * added sdk test command * migrate SSX functions to UserAuthroization resource * updated UserAuthorization logic + tests * update tests * renamed resources to modules * setup SSX to import modules * include modules in ssx * correct typescript errors * add module dependencies to tests * cleanup * fix provider reference * remove ssx init + connected

* add signature encryption module * update tests * text box for ciphertext * made changes to extension to fix tests --------- Co-authored-by: franklovefrank <[email protected]>

* Signature Encryption Module (spruceid#113) * add signature encryption module * update tests * text box for ciphertext * made changes to extension to fix tests --------- Co-authored-by: franklovefrank <[email protected]> * added storage + datavault module * added datavault * added ssx-notepad-datavault * move from blob to json * encryption debugging changes * fix build error * updated example to use only datavault instead of server * updated storage module to use indexedDB instead of localstorage * prevent occasional install bug * update indexeddb usage * updated notepad example * remove examples from workspace due to installation issues * added helmet back for walleconnect * added mock indexeddb * cleanup * add missing dependencies for test * updated docstrings * Add examples to monorepo and fix notepad example build * Fix getSigner on tests * configurable store name * fixed tests --------- Co-authored-by: franklovefrank <[email protected]> Co-authored-by: Juliano <[email protected]>

Juliano1612 · 2023-04-25T17:52:36Z

packages/ssx-sdk/tests/modules/Encryption.test.ts

    test('Should be able to encrypt and decrypt a message', async () => {
      const message = 'Hello World';
-      const encryptedMessage = await encryption.encrypt(message);
-      const decryptedMessage = await encryption.decrypt(encryptedMessage);
-      expect(decryptedMessage).toEqual(message);
+      const encryptedData = await encryption.encrypt({
+        content: message,
+        accessControlConditions: LIT_ENCRYPTION_ACCESS_CONTROL_CONDITIONS
+      });
+      const decryptedData = await encryption.decrypt({
+        ...encryptedData,
+        accessControlConditions: LIT_ENCRYPTION_ACCESS_CONTROL_CONDITIONS
+      });
+      expect(decryptedData.decryptedString).toEqual(message);


I updated the tests here, but I still have trouble with the crypto library. Left them disabled.

Juliano1612 · 2023-04-25T17:53:23Z

packages/ssx-sdk/src/ssx.ts

+    if (
+      this.config?.modules?.encryption === true ||
+      (this.config?.modules?.encryption as SSXEncryptionModuleConfig)?.module === 'SignatureEncryption'
+    ) {
+      this.encryption = new SignatureEncryption({}, this.userAuthorization);
+    } else if ((this.config?.modules?.encryption as SSXEncryptionModuleConfig)?.module === 'LitEncryption') {
+      this.encryption = new LitEncryption({}, this.userAuthorization);
+    }


Just a temporary approach until we define the way we configure the modules. I'm assuming SignatureEncryption as default

Juliano1612 · 2023-04-25T17:54:37Z

packages/ssx-sdk/src/modules/Encryption.ts

+   * @param chainId - Unique chain id.
+   * @returns - The chain name follwing the Lit Protocol docs.
+   */
+  private getChainName = (chainId: number) => {


Had to create this to translate chainId -> chain string followingLit supported chains.

Juliano1612 · 2023-04-25T17:54:53Z

packages/ssx-sdk/src/modules/Encryption.ts

+      // case ?:  
+      //   return "cosmos";
+      // case ?:  
+      //   return "kyve";
+      // case ?:  
+      //   return "evmosCosmos";
+      // case ?:  
+      //   return "evmosCosmosTestnet";
+      // case ?:  
+      //   return "cheqd";
+      // case ?:  
+      //   return "juno"; 


Couldn't find these chainIds

Juliano1612 · 2023-04-25T17:59:03Z

packages/ssx-sdk/src/modules/Encryption.ts

@@ -1,5 +1,6 @@
 import * as jose from 'jose';
-import { IUserAuthorization } from './UserAuthorization';
+import { IUserAuthorization, UserAuthorization } from './UserAuthorization';
+import * as LitJsSdk from '@lit-protocol/lit-node-client';


I'm receiving this warning after the compilation:

WARNING in ../../node_modules/@lit-protocol/ecdsa-sdk/src/lib/ecdsa-sdk.js 428:22-61 Critical dependency: the request of a dependency is an expression @ ../../node_modules/@lit-protocol/ecdsa-sdk/src/index.js 4:21-47 @ ../../node_modules/@lit-protocol/crypto/src/lib/crypto.js 7:18-52 @ ../../node_modules/@lit-protocol/crypto/src/index.js 4:21-44 @ ../../node_modules/@lit-protocol/lit-node-client-nodejs/src/lib/lit-node-client-nodejs.js 9:17-48 @ ../../node_modules/@lit-protocol/lit-node-client-nodejs/src/index.js 6:29-68 18:21-60 @ ../../node_modules/@lit-protocol/lit-node-client/src/index.js 25:21-68 @ ./src/modules/Encryption.ts 37:30-70 @ ./src/modules/index.ts 20:13-36 @ ./src/index.ts 19:13-33

And this is generating a warning on ssx-test-dapp:

Compiled with warnings. Critical dependency: require function is used in a way in which dependencies cannot be statically extracted Search for the keywords to learn more about each warning. To ignore, add // eslint-disable-next-line to the line before. WARNING in ../../packages/ssx-sdk/dist/index.js 59504:80-87 Critical dependency: require function is used in a way in which dependencies cannot be statically extracted webpack compiled with 1 warning

@skgbafa or @w4ll3 have you ever seen this?

Juliano1612 · 2023-04-26T15:32:52Z

packages/ssx-sdk/src/modules/Encryption.ts

+      signedMessage: session.siwe,
+    };
+
+    const { encryptedString, symmetricKey } = await LitJsSdk.encryptString(data.content)


I'm only encrypting strings here, but they have other methods to encrypt. We can add/change this call

skgbafa and others added 10 commits March 31, 2023 14:50

Signature Encryption Module (spruceid#113)

f13c729

* add signature encryption module * update tests * text box for ciphertext * made changes to extension to fix tests --------- Co-authored-by: franklovefrank <[email protected]>

Add LitEncryption class

57e578c

Fix webpack issue

db8065e

Fix Jest issue

32763fd

Update ssx test dapp

063a5a0

Improve docstrings

1f7e843

Fix tests

ae3e2db

Update Lit tests

5a7a2f5

Juliano1612 commented Apr 25, 2023

View reviewed changes

Juliano1612 requested review from skgbafa and w4ll3 April 25, 2023 18:00

Cleanups

f01b098

Juliano1612 commented Apr 26, 2023

View reviewed changes

Juliano1612 changed the title ~~LitEncryption Module~~ [WIP] LitEncryption Module May 3, 2023

skgbafa force-pushed the feat/ssx-modules branch from 1ec7568 to 1000806 Compare May 11, 2023 19:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[WIP] LitEncryption Module #130

[WIP] LitEncryption Module #130

Juliano1612 commented Apr 25, 2023 •

edited by skgbafa

Loading

Juliano1612 Apr 25, 2023 •

edited

Loading

Juliano1612 Apr 25, 2023

Juliano1612 Apr 25, 2023

Juliano1612 Apr 25, 2023

Juliano1612 Apr 25, 2023

Juliano1612 Apr 26, 2023

[WIP] LitEncryption Module #130

Are you sure you want to change the base?

[WIP] LitEncryption Module #130

Conversation

Juliano1612 commented Apr 25, 2023 • edited by skgbafa Loading

Description

Type

Diligence Checklist

Juliano1612 Apr 25, 2023 • edited Loading

Choose a reason for hiding this comment

Juliano1612 Apr 25, 2023

Choose a reason for hiding this comment

Juliano1612 Apr 25, 2023

Choose a reason for hiding this comment

Juliano1612 Apr 25, 2023

Choose a reason for hiding this comment

Juliano1612 Apr 25, 2023

Choose a reason for hiding this comment

Juliano1612 Apr 26, 2023

Choose a reason for hiding this comment

Juliano1612 commented Apr 25, 2023 •

edited by skgbafa

Loading

Juliano1612 Apr 25, 2023 •

edited

Loading