search

Luminarys / synapse

Synapse BitTorrent Daemon
https://synapse-bt.org
ISC License
853 stars 48 forks source link

Use rustls with sycli #193

Closed Luminarys closed 4 years ago

Luminarys commented 4 years ago

Synapse will have openssl completely removed as a dependency soon. Ideally sycli should also have it removed. I see two viable approaches:

  1. Modify dependencies to remove it. This seems not so bad for reqwest, but quite difficult for tungstenite. Some more investigation will be needed here
  2. Extract out websocket parsing code from synapse and allow it to be used by sycli. This should probably be preferred even though it will mean more work since it reduces dependencies (sycli's binary is massive as is).
Luminarys commented 4 years ago

This was not too bad of a change. We're now OpenSSL free and sycli's build is now 4.8 MB.