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.

Sign up for GitHub

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

Jump to bottom

client: add WebRTC transport #1

Closed
wants to merge 26 commits into from
Closed

client: add WebRTC transport #1

wants to merge 26 commits into from

Conversation

melekes
Copy link
Owner

@melekes melekes commented Jun 30, 2022 •
edited
Loading

@melekes

This comment was marked as resolved.

Sign in to view
@melekes

This comment was marked as outdated.

Sign in to view
@mxinden
Copy link

mxinden commented Jul 11, 2022

I think it’s because webrtc-rs wasn’t designed for multiple connections between two parties with same certificates.

Do I understand correctly that implementations have no way to differentiate incoming data by connection given that both endpoints use the same certificate and the same UDP port @melekes?

@melekes
Copy link
Owner Author

melekes commented Jul 11, 2022

I think it’s because webrtc-rs wasn’t designed for multiple connections between two parties with same certificates.

Do I understand correctly that implementations have no way to differentiate incoming data by connection given that both endpoints use the same certificate and the same UDP port @melekes?

the same certificate and the same UDP port and the same ICE credentials (username/password, both equal to DTLS fingerprint). I haven't tried the scenario with different ICE credentials.

@mxinden
Copy link

mxinden commented Jul 11, 2022

I haven't tried the scenario with different ICE credentials.

Would be great in case that would allow multiple connections between two peers. From what I can tell based on libp2p/specs#412 we could use a random string for username+password in the initial STUN for each connection, right?

@melekes
Copy link
Owner Author

melekes commented Jul 11, 2022

If we disable fingerprint verification on the server (libp2p/specs#412 (comment)), sure.

@melekes

This comment was marked as resolved.

Sign in to view
ggwpez and others added 9 commits October 10, 2022 14:25
@ggwpez
Dont ignore errors in pallet benchmarking (paritytech#12449)
df81976 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@rcny
Update UI tests for Rust 1.64 (paritytech#12440)
9672c36 
* Update UI tests for Rust 1.64

* Test with the staging image

* Switch back to production
@lexnv @niklasad1 @jsdw
rpc: Implement transaction RPC API (paritytech#12328)
488fc24 
* rpc/tx: Add transaction structures for serialization

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Add public facing `TransactionEvent`

To circumvent the fact that serde does not allow mixing
`#[serde(tag = "event")]` with
`#[serde(tag = "event", content = "block")]`
the public facing subscription structure is serialized
and deserialized to an intermmediate representation.

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Add trait for the `transaction` API

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Convert RPC errors to transaction events

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Implement `transaction` RPC methods

Signed-off-by: Alexandru Vasile <[email protected]>

* tx-pool: Propagate tx index to events

Signed-off-by: Alexandru Vasile <[email protected]>

* tx-pool: Adjust testing to reflect tx index in events

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Convert tx-pool events for the new RPC spec

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Convert tx-pool `FinalityTimeout` event to `Dropped`

Signed-off-by: Alexandru Vasile <[email protected]>

* service: Enable the `transaction` API

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Add tests for tx event encoding and decoding

Signed-off-by: Alexandru Vasile <[email protected]>

* tx: Add indentation for subscriptions

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Fix documentation

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Serialize usize to hex

Signed-off-by: Alexandru Vasile <[email protected]>

* tx-pool: Rename closure parameters

Signed-off-by: Alexandru Vasile <[email protected]>

* service: Separate RPC spec versions

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Use `H256` for testing block's hash

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Serialize numbers as string

Signed-off-by: Alexandru Vasile <[email protected]>

* tx-pool: Backward compatibility with RPC v1

Signed-off-by: Alexandru Vasile <[email protected]>

* Update client/rpc-spec-v2/src/transaction/transaction.rs

Co-authored-by: Niklas Adolfsson <[email protected]>

* rpc/tx: Remove comment about serde clone

Signed-off-by: Alexandru Vasile <[email protected]>

* rpc/tx: Use RPC custom error code for invalid tx format

Signed-off-by: Alexandru Vasile <[email protected]>

* Update client/rpc-spec-v2/src/transaction/event.rs

Co-authored-by: James Wilson <[email protected]>

* rpc/tx: Adjust internal structures for serialization/deserialization

Signed-off-by: Alexandru Vasile <[email protected]>

Signed-off-by: Alexandru Vasile <[email protected]>
Co-authored-by: Niklas Adolfsson <[email protected]>
Co-authored-by: James Wilson <[email protected]>
@tomaka
Remove the unused light client requests (paritytech#12470)
60be7b8 
* Remove the unused light client requests

* Add comment about new ids
@altonen
Fix flaky service test (paritytech#12472)
6072b90 
Sometimes `NotificationStreamOpenened` would be received for the
other protocol before `SyncConnected` was received so when the
connection was closed, an incorrect event was read from the event
stream.
@KiChjang
Rename from_components to from_parts (paritytech#12473)
1bf2e6d 
* Rename `from_components` to `from_parts`

* Fixes

* Spelling
@shawntabrizi
Avoid Unstable Sort (paritytech#12455)
ccc8f6c 
* dont use unstable sort

* remove comment

* add clippy rule
@michalkucharczyk @andresilva
@davxy @bkchr
Finalized block event triggers tx maintanance (paritytech#12305)
94b731c 
* finalized block event triggers tx maintanance

* tx-pool: enactment helper introduced

* tx-pool: ChainApi: added tree_route method

* enactment logic implemented + tests

Signed-off-by: Michal Kucharczyk <[email protected]>

* Some additional tests

* minor improvements

* trigger CI job

* fix compilation errors

ChainApi::tree_route return type changed to Result<Option<..>>, as some
implementations (tests) are not required to provide this tree route.

* formatting

* trait removed

* implementation slightly simplified

(thanks to @koute)

* get rid of Arc<> in EnactmentState return value

* minor improvement

* Apply suggestions from code review

Co-authored-by: André Silva <[email protected]>

* Apply suggestions from code review

* comment updated + formatting

* Apply suggestions from code review

Co-authored-by: André Silva <[email protected]>
Co-authored-by: Davide Galassi <[email protected]>

* formatting

* finalization notification bug fix

+ new test case
+ log::warn message when finalized block is being retracted by new event

* added error message on tree_route failure

* Apply suggestions from code review

Co-authored-by: Bastian Köcher <[email protected]>

* use provided tree_route in Finalized event

* Option removed from ChainApi::tree_route

* doc added, test and logs improved

* handle_enactment aligned with original implementation

* use async-await

* Apply suggestions from code review

Co-authored-by: André Silva <[email protected]>

* Apply suggestions from code review

Co-authored-by: Bastian Köcher <[email protected]>

* formatting + warn->debug

* compilation error fix

* enactment_state initializers added

* enactment_state: Option removed

* manual-seal: compilation & tests fix

* manual-seal: tests fixed

* tests cleanup

* another compilation error fixed

* TreeRoute::new added

* get rid of pub hack

* one more test added

* formatting

* TreeRoute::new doc added + formatting

* Apply suggestions from code review

Co-authored-by: Davide Galassi <[email protected]>

* (bool,Option) simplified to Option

* log message improved

* yet another review suggestions applied

* get rid of hash in handle_enactment

* Apply suggestions from code review

Co-authored-by: Bastian Köcher <[email protected]>

* Update client/transaction-pool/src/lib.rs

Co-authored-by: Bastian Köcher <[email protected]>

* minor corrections

* EnactmentState moved to new file

* File header corrected

* error formatting aligned with codebase

* Apply suggestions from code review

Co-authored-by: Bastian Köcher <[email protected]>

* remove commented code

* small nits

Signed-off-by: Michal Kucharczyk <[email protected]>
Co-authored-by: André Silva <[email protected]>
Co-authored-by: Davide Galassi <[email protected]>
Co-authored-by: Bastian Köcher <[email protected]>
Co-authored-by: André Silva <[email protected]>
@michalkucharczyk
tx-pool: failing tests fixed (paritytech#12481)
88db102
@mxinden
Copy link

mxinden commented Oct 12, 2022

which themselves depend on https://github.com/libp2p/rust-libp2p/pull/2931/files.

Sorry for the delay here. I am on it :)

tomaka and others added 13 commits October 12, 2022 13:01
@tomaka
Clarify the "direction" field of block requests (paritytech#12438)
06a9f0a
@michalkucharczyk
BlockId::Number refactor: trivial changes to BlockId::Hash (paritytec…
f8a0b7a 
…h#12471)

* Trivial BlockId::Number => Hash

* missed BlockId::Hash added
@shawntabrizi
Dont use benchmark range on constant functions (paritytech#12456)
b324e51 
* dont use benchmark range on constant function

* update weights

* fix

* new weights

* Update frame/examples/basic/src/benchmarking.rs

Co-authored-by: parity-processbot <>
@dmitry-markin @andresilva
Punish peers for duplicate GRANDPA neighbor messages (paritytech#12462)
94941a8 
* Decrease peer reputation for duplicate GRANDPA neighbor messages.

* Fix comparison

* Fix update_peer_state() validity condition

* Add negative test

* Rework update_peer_state() validity condition, add tests

* update_peer_state() validity condition: invert comparison

* Split InvalidViewChange and DuplicateNeighborMessage misbehaviors

* Enforce rate-limiting of duplicate GRANDPA neighbor packets

* Update client/finality-grandpa/src/communication/gossip.rs

Co-authored-by: André Silva <[email protected]>

* Make rolling clock back in a test safer

Co-authored-by: André Silva <[email protected]>
@altonen @bkchr
Introduce mockable ChainSync object for testing (paritytech#12480)
983b6b0 
* Introduce mockable `ChainSync` object for testing

`mockall` allows to mock `ChainSync` and to verify that the calls made
to `ChaiSync` are firstly executed at all, that they're executed in
correct order and with correct parameters.

This allows to verify, e.g., that delegating calls directly to
`ChainSync` from `NetworkService` still calls the correct functions with
correct arguments even if `Protocol` middleman is removed.

* Add Cargo.lock

* Fix tests

* Update client/network/Cargo.toml

Co-authored-by: Bastian Köcher <[email protected]>

* Update Cargo.lock

* Fix clippy and documentation

Co-authored-by: Bastian Köcher <[email protected]>
Co-authored-by: parity-processbot <>
@Szegoo @Lederstrumpf @acatangiu
pallet-mmr: RPC API and Runtime API work with block numbers (parityte…
1f39c90 
…ch#12345)

* pallet-mmr: RPC API works with block_numbers

* fixes

* update rpc

* fmt

* final touches in the rpc

* temporary fix

* fix

* fmt

* docs

* Update lib.rs

* use NumberFor

* validate input

* update runtime

* convert block_number to u64

* small edit

* update runtime api

* test fix

* runtime fix

* update test function

* fmt

* fix nits

* remove block_num_to_leaf_index from runtime api

* Update frame/merkle-mountain-range/src/lib.rs

Co-authored-by: Robert Hambrock <[email protected]>

* fix tests

* get the code to compile after merge

* get the tests to compile

* fix in tests?

* fix test

* Update frame/merkle-mountain-range/src/tests.rs

Co-authored-by: Adrian Catangiu <[email protected]>

* Update frame/merkle-mountain-range/src/lib.rs

Co-authored-by: Adrian Catangiu <[email protected]>

* Update primitives/merkle-mountain-range/src/lib.rs

Co-authored-by: Adrian Catangiu <[email protected]>

* fix errors & nits

* change block_num_to_leaf_index

* don't make any assumptions

* Update frame/merkle-mountain-range/src/tests.rs

Co-authored-by: Adrian Catangiu <[email protected]>

* Update frame/merkle-mountain-range/src/tests.rs

Co-authored-by: Adrian Catangiu <[email protected]>

* Update frame/merkle-mountain-range/src/tests.rs

Co-authored-by: Adrian Catangiu <[email protected]>

* fix

* small fix

* use best_known_block_number

* best_known_block_number instead of leaves_count

* more readable?

* remove warning

* Update frame/merkle-mountain-range/src/lib.rs

Co-authored-by: Robert Hambrock <[email protected]>

* simplify

* update docs

* nits

* fmt & fix

* merge fixes

* fix

* small fix

* docs & nit fixes

* Nit fixes

* remove leaf_indices_to_block_numbers()

* fmt

Co-authored-by: Robert Hambrock <[email protected]>
Co-authored-by: Adrian Catangiu <[email protected]>
@koute
Enable the wasmtime-based WASM executor by default (paritytech#12486)
fd00b14
@michalkucharczyk
Trivial BlockId::Number => Hash (paritytech#12490)
6e04e48
@michalkucharczyk
BlockId removal: refactor: Backend::state_at (paritytech#12488)
f313987 
* Minor naming improved

* BlockId removal refactor: Backend::state_at

* formatting
@ggwpez
Try-runtime CLI fix weight parsing (paritytech#12491)
0ee0327 
Signed-off-by: Oliver Tale-Yazdi <[email protected]>

Signed-off-by: Oliver Tale-Yazdi <[email protected]>
@kvinwang
Export fn code_hash (paritytech#12479)
6f453b5 
Co-authored-by: parity-processbot <>
@altonen
Introduce ChainSyncInterface (paritytech#12489)
30a7a5b 
* Introduce `ChainSyncInterface`

`ChainSyncInterface` provides an asynchronous interface for other
subsystems to submit calls to `ChainSync`. This allows `NetworkService`
to delegate calls to `ChainSync` while still providing the same API
for other subsystems (for now). This makes it possible to move the
syncing code in piecemeal fashion out of `protocol.rs` as the calls
are just forwarded to `ChainSync`.

* Apply review comments

* Fix tests
@bkchr
sp-api: Remove invalid unsafe trait bounds (paritytech#12502)
951f075 
The runtime api implementation contained invalid unsafe trait bounds. `Sync` was never correct there
and `Send` should have not been "force implemented".
acatangiu and others added 4 commits October 17, 2022 09:09
@acatangiu
pallet-mmr: fix offchain db for sync from zero (paritytech#12498)
273e264 
* pallet-mmr: cosmetic improvements

* pallet-mmr: fix offchain storage for initial sync

* address review comments

* pallet-mmr: change offchain fork-resistant key to `(prefix, pos, parent_hash)`

Do this so that both canon and fork-resitant keys have the same
`(prefix, pos).encode()` prefix. Might be useful in the future if we'd
be able to to "get" offchain db entries using key prefixes as well.

Signed-off-by: acatangiu <[email protected]>

Signed-off-by: acatangiu <[email protected]>
@omahs
Fix: typo (paritytech#12505)
acacb53 
Fix: typo
@dmitry-markin @altonen
Upgrade libp2p to 0.49.0 (paritytech#12256)
444cf9d 
* cargo upgrade libp2p

* Get rid of `NetworkBehaviourEventProcess` in handling of `CustomMessageOutcome`

* Get rid of `NetworkBehaviourEventProcess` in handling of `request_responses::Event`

* Get rid of `NetworkBehaviourEventProcess` in handling of `peer_info::PeerInfoEvent`

* Get rid of `NetworkBehaviourEventProcess` in handling of `DiscoveryOut`

* Get rid of `poll()` method in `Bahaviour`

* minor: comments

* Upgrade libp2p to 0.49.0 (unreleased)

* Support multiple Kad protocol names

* Make borrow checker happy

* minor: wording

* Make substrate build with libp2p-0.49.0

* rustfmt

* Get rid of MdnsWrapper

* Resolve deprecation warnings

* Fix documentation

* Apply suggestions from code review: fix typos

Co-authored-by: Aaro Altonen <[email protected]>

* Apply suggestion: simplify kad protocol name matching

Co-authored-by: Aaro Altonen <[email protected]>
@melekes
client: add WebRTC transport
fb707c8 
Refs paritytech/smoldot#1712
@melekes melekes marked this pull request as ready for review October 18, 2022 12:21
@melekes melekes closed this Oct 18, 2022
@melekes melekes reopened this Oct 18, 2022
@melekes melekes closed this Oct 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.