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

We now depend on native openssl again #5562

Open
2 tasks done
gilescope opened this issue Sep 3, 2024 · 1 comment
Open
2 tasks done

We now depend on native openssl again #5562

gilescope opened this issue Sep 3, 2024 · 1 comment
Labels
I2-bug The node fails to follow expected behavior. I10-unconfirmed Issue might be valid, but it's not yet known.

Comments

@gilescope
Copy link
Contributor

Is there an existing issue?

  • I have searched the existing issues

Experiencing problems? Have you tried our Stack Exchange first?

  • This is not a support question.

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 ( algesten/str0m#326 ).

Steps to reproduce

cargo build --workspace

@gilescope gilescope added I10-unconfirmed Issue might be valid, but it's not yet known. I2-bug The node fails to follow expected behavior. labels Sep 3, 2024
@burdges
Copy link

burdges commented Sep 3, 2024

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
I2-bug The node fails to follow expected behavior. I10-unconfirmed Issue might be valid, but it's not yet known.
Projects
None yet
Development

No branches or pull requests

2 participants