search

paritytech / polkadot-sdk

The Parity Polkadot Blockchain SDK
https://polkadot.network/
1.78k stars 639 forks source link

We now depend on native openssl again #5562

Open gilescope opened 1 week ago

gilescope commented 1 week ago

Is there an existing issue?

Experiencing problems? Have you tried our Stack Exchange first?

Description of bug

Unfortunately we moved to str0m and now polkadot-sdk depends on the native openssl to compile:

This reduces the set of computers where polkadot-sdk compiles without intervention. We've done well to get off openssl by default. It would be a shame to have to pick up having to configure this native dependency again across all machines that build substrate in the world.

Either we should move away from str0m or find some way that we can use pure rust crypto with it ( https://github.com/algesten/str0m/issues/326 ).

Steps to reproduce

cargo build --workspace

burdges commented 1 week ago

Do all build configurations require WebRTC? Appears WebRTC has made progress towards TLS 1.3, like Mozilla closed their issue for WebRTC on TLS 1.3, so maybe DTLS 1.2 could be depricated for our use cases, although that's a pretty big drop, which could bring much chaos.