Skip to content
This repository has been archived by the owner on Feb 17, 2025. It is now read-only.

Feature/json rpc ws #1445

Merged
merged 28 commits into from
Dec 20, 2022
Merged

Feature/json rpc ws #1445

merged 28 commits into from
Dec 20, 2022

Conversation

tclemos
Copy link
Contributor

@tclemos tclemos commented Dec 7, 2022

Closes #1408.

What does this PR do?

Adds support to Web Sockets to the RPC.

Reviewers

@cla-bot cla-bot bot added the cla-signed label Dec 7, 2022
@tclemos tclemos self-assigned this Dec 7, 2022
@tclemos tclemos marked this pull request as ready for review December 12, 2022 20:55
@tclemos tclemos requested review from kind84 and Psykepro and removed request for ARR552 and cool-develope December 14, 2022 19:26
Copy link
Contributor

@kind84 kind84 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM terrific job @tclemos 👏

@tclemos
Copy link
Contributor Author

tclemos commented Dec 20, 2022

I'll merge this PR even with the e2e-group1 test failing because this is already failing in the develop branch and seems to be a mismatch between the image we have for l1 mock network and mock prover when running the tests.

we need to fix this on develop branch. Issue #1470 was created to track this problem.

@tclemos tclemos merged commit 34263ed into develop Dec 20, 2022
@tclemos tclemos deleted the feature/json-rpc-ws branch December 20, 2022 16:24
@tclemos tclemos restored the feature/json-rpc-ws branch January 25, 2023 15:12
@tclemos tclemos deleted the feature/json-rpc-ws branch January 25, 2023 15:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add websocket endpoints to the API
2 participants