Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

tee-worker is blocked when create node_api if parachain don't start #1474

Open
DogLi opened this issue Mar 16, 2023 · 1 comment
Open

tee-worker is blocked when create node_api if parachain don't start #1474

DogLi opened this issue Mar 16, 2023 · 1 comment
Labels
D2-bug Something isn't working I2-medium should be completed within 10 working days Stale

Comments

@DogLi
Copy link
Contributor

DogLi commented Mar 16, 2023

the WsRpcClient here include a RpcClient, and RpcClient impl Handle, the Handle includes a function on_error, it uses default on_error function which just print out the error log. So when the WsRpcClient call the function direct_rpc_request, it fist connect the server, when getting a fused error, just print out the error log, the connect function still returns Ok, the result_out.recv() in the direct_rpc_request will waiting forever.

need to test:

When node_api is created successfully(which means the parachain is ok), if the WebSocket disconnected, what will happen when we use node_api? will it block the whole program?

maybe some solutions:

  1. How to re-establish the connection after listen event connection is lost? scs/substrate-api-client#246
  2. use tungstenite-client feature
  3. Software Design: Add reconnection possibility scs/substrate-api-client#382
@jingleizhang jingleizhang added D2-bug Something isn't working I2-medium should be completed within 10 working days labels Mar 22, 2023
@github-actions
Copy link
Contributor

❗ This issue is stale because it has been open for 60 days with no activity.
Remove Stale label or update it, otherwise this issue will be closed in 7 days.
@litentry/parachain

@github-actions github-actions bot added the Stale label May 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D2-bug Something isn't working I2-medium should be completed within 10 working days Stale
Projects
None yet
Development

No branches or pull requests

2 participants