crunchy-labs / crunchy-cli

👇 Command-line downloader for Crunchyroll
MIT License
606 stars 62 forks source link

Unknown TLS backend passed to `use_preconfigured_tls` #437

Closed Ziedelth closed 5 months ago

Ziedelth commented 5 months ago

Describe the bug When I try to use a command who need credentials :

thread 'main' panicked at crunchy-cli-core/src/lib.rs:402:34:
called `Result::unwrap()` on an `Err` value: reqwest::Error { kind: Builder, source: "Unknown TLS backend passed to `use_preconfigured_tls`" }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

To Reproduce Steps / command to reproduce the behavior:

Dockerfile

FROM rust as rust

# Install ffmpeg
RUN apt-get update && apt-get install -y ffmpeg

RUN git clone https://github.com/crunchy-labs/crunchy-cli.git
WORKDIR /crunchy-cli
RUN cargo build --release
RUN cargo install --force --path .

RUN mkdir /crunchy-cli/data
WORKDIR /crunchy-cli/data

ENTRYPOINT ["crunchy-cli"]
$ docker run -it  --mount type=bind,source="$(pwd)",target=/crunchy-cli/data --rm --name lytecrh lytecrh:latest --credentials "username:password" download -o DAL_S5EP08.mp4 -a ja-JP -s fr-FR https://www.crunchyroll.com/fr/watch/G9DUE0Z35/the-one-who-pulled-the-trigger
benkrej commented 5 months ago

Hi, what was the solution to this issue?

benkrej commented 5 months ago

Fixed after aligning rustls version to reqwests internal rustls version

enterprisey commented 4 months ago

I have the same error even on the latest git version (commit 756022b955633c73c2920091b3f02d37ab791004). I have verified that the versions specified in the fix commit (commit 509683d23a5689dc958a79440227585c9d383c30) have been downloaded (tokio 1.38 and not so native tls c7ac566).

enterprisey commented 4 months ago

Update, only the version installed with cargo install --git https://github.com/crunchy-labs/crunchy-cli crashed like this; checking out the repo myself and running it worked fine. So maybe the issue's been fixed.