Use native TLS only, don't mix it with rustls

reqwest uses native TLS, but rspotify with ureq currently uses rustls, i.e. ncspot bundles rustls and is linked with system TLS library at the same time. This has already been fixed in 80da5a8, but this commit has been reverted in aeff120 due to a bug in rspotify that was fixed in 0.13.1 (ramsayleung/rspotify#471).
hrkfdn · Apr 2, 2024 · 5a5e8b7 · 5a5e8b7
1 parent 7b3b541
commit 5a5e8b7
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 69 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - All requests are correctly proxied when the relevant environment variables are set
+- `ncspot` binary is simultaneously linked with the system TLS library (native-tls) and Rustls - only native-tls is used now
 
 ## [1.1.0] - 2024-03-05
 

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -73,7 +73,7 @@ signal-hook = "0.3.0"
 
 [dependencies.rspotify]
 default-features = false
-features = ["client-ureq", "ureq-rustls-tls"]
+features = ["client-ureq", "ureq-native-tls"]
 version = "0.13.1"
 
 [dependencies.cursive]