Rework `Worker` api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

clangenb · 2024-04-06T05:56:28Z

This PR reworks the worker api into the new design implemented in the worker, which moved the TLS endpoint to inside the enclave, and has changed the API in general. Closes #86, #90. Additionally, it lays the foundation of separate worker implementation by distinguishing between the overarching base Worker implementing the IWorker interface. Then we have two different worker implementations, the IntegriteeWorker and the EncointerWorker. Although, some functions are not generic enough to support both yet, and hence the EncointerWorker is not functional, as it uses types for the IntegriteeWorker, tracking issue #91.

Working

Handle self-signed certificate in node-js tests
get shielding key
get shard vault
trusted balance getter
state_executeGetter: trusted nonce getter, trusted balance getter
author_submitExrinsic: trustedBalanceTransfer, balanceUnshield

Note

There is no support for subscribing to results of a trusted call with author_submitAndWatchExtrinsic

…accounts

This reverts commit 3768996.

…l changes needed.

…e shielding key

…once fails currently)

clangenb · 2024-04-10T02:57:34Z

packages/worker-api/src/parsers.ts

+      // Enforce using the pure javascript implementations by
+      // setting the `browser` environment, as compatibility
+      // with node's crypto is broken and leads to bad outputs.
+      environment: 'browser',


This fixes #90

clangenb added 20 commits March 29, 2024 15:26

[worker] test simple websocket connection

5c70bd2

[jest] allow any kind of certificate

e7ba030

[worker] update get balance to just get regular balance of incognito …

8d32315

…accounts

[network] update localDockerNetwork to be correct

e49ed8a

[worker.spec] successfully connect to the worker.

3768996

Revert "[worker.spec] successfully connect to the worker."

9b75854

This reverts commit 3768996.

[worker.spec] connect to worker with original implementation; protoco…

641858a

…l changes needed.

[worker] implement new trusted rpc protocol and successfully fetch th…

a5a2222

…e shielding key

[worker.spec] remove primitive websocket test

4125540

[worker.spec] separate tests for encointer-specific protocol

f0409f1

[worker] successfully get free balance of an account

32dba38

[worker] implement getting shard vault

3d4e352

[worker] fix return value parsing in generic cases

b7f3b26

[worker] refactor request creation into its own file

767b5d8

[worker] rename getter api to sendRequest

de9c307

[worker] update public getter

1461d73

[worker] rename callGetter to sendRequest

992cfea

[worker] more explicit naming

86a9af5

[worker] implement draft of trusted call balance transfer, (getting n…

1966fe9

…once fails currently)

[worker] fix getting nonce by using toHex instead of to u8a

efb5ff8

clangenb marked this pull request as draft April 6, 2024 05:56

clangenb added 5 commits April 6, 2024 17:00

[worker] fix signature type in trusted calls/getters

b1b5c6e

[worker] fix interpreting the result of getters

7773a51

[worker] fix signature of trusted call

187075e

[worker] try to encrypt the trusted call upon sending

e13865a

[worker] introduce trusted operation

7efd96a

clangenb mentioned this pull request Apr 8, 2024

Node-RSA seems to create output violating the standards #90

Closed

clangenb added 3 commits April 8, 2024 11:38

[worker] fix encryption

f1e8790

[worker] add comment for node-rsa setting

90e2c4f

[worker] successfully send a top to the worker.

13066b5

clangenb added 5 commits April 8, 2024 13:38

[worker] better doc

823659d

[worker] fix sending trusted calls with author_submitExtrinsic

46f550d

[jest] remove tls certificate config that didn't work

e22849b

[jest] remove tls certificate config that didn't work

911ca61

[worker] implement balance_unshield trusted call

29473cc

clangenb mentioned this pull request Apr 8, 2024

demonstrate incognitee with js client integritee-network/worker#1593

Closed

clangenb added 7 commits April 10, 2024 08:59

[worker] use charlie in tests

a9fa8a0

[worker] rename interface IEncointerWorker to IIntegriteeWorker

c1068af

[worker] separate encointer from integritee worker

aa14787

[worker] finalize separation of workers

6a846f3

[worker] separate types of integritee and encointer

57ab509

[worker] fix unused imports in encointer worker

2c11c03

[worker] skip CI that needs a worker

f21fad8

clangenb changed the title ~~Incognitee Support~~ Rework Worker api and make it work against recent integritee-worker (aka tls directly into the enclave) Apr 10, 2024

clangenb mentioned this pull request Apr 10, 2024

Reintroduce Worker implementation for encointer #91

Open

[worker] add todo for encointer worker

ab2e0d8

clangenb marked this pull request as ready for review April 10, 2024 02:56

clangenb merged commit 5d9a714 into master Apr 10, 2024
2 of 3 checks passed

clangenb commented Apr 10, 2024

View reviewed changes

clangenb mentioned this pull request Apr 10, 2024

Fix inconsistent use of CommunityIdentifier vs. ShardVault #93

Closed

clangenb deleted the cl/incognitee-support branch April 17, 2024 01:25

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Rework `Worker` api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

Rework `Worker` api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

clangenb commented Apr 6, 2024 •

edited

Loading

clangenb Apr 10, 2024

Rework Worker api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

Rework Worker api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

Conversation

clangenb commented Apr 6, 2024 • edited Loading

clangenb Apr 10, 2024

Choose a reason for hiding this comment

Rework `Worker` api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

Rework `Worker` api and make it work against recent integritee-worker (aka tls directly into the enclave) #89

clangenb commented Apr 6, 2024 •

edited

Loading