Skip to content

chore(deps): align external dependencies with SMG - #7

Merged
BugenZhao merged 2 commits into
mainfrom
bz/align-smg-dependencies
Jul 14, 2026
Merged

chore(deps): align external dependencies with SMG#7
BugenZhao merged 2 commits into
mainfrom
bz/align-smg-dependencies

Conversation

@BugenZhao

@BugenZhao BugenZhao commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • align the direct reqwest dependency with SMG at 0.13.4
  • align the blake3 minimum version with SMG at 1.8
  • forward the selected TLS backend to both direct HTTP requests and optional hf-hub requests
  • make Rustls the default TLS backend while retaining native TLS as an explicit opt-in

Why

MediaConnector::new accepts a reqwest::Client, so the standalone crate's reqwest 0.12 dependency was incompatible with SMG's reqwest 0.13 client. Reqwest 0.13 also renamed its Rustls feature to rustls; explicitly forwarding each TLS choice to hf-hub keeps model-config downloads functional now that hf-hub retains its own reqwest 0.12 dependency.

SMG uses Rustls for outbound HTTP, so selecting rustls-tls by default keeps the standalone crate's default behavior aligned with its primary downstream. Consumers that require native TLS can select native-tls with default features disabled.

Downstream callers constructing MediaConnector with a reqwest client should use reqwest 0.13 with this revision.

Validation

  • cargo test --all-targets
  • cargo test --all-targets --no-default-features
  • cargo clippy --all-targets -- --deny warnings
  • cargo clippy --all-targets --no-default-features -- --deny warnings
  • cargo check --all-targets --no-default-features --features hf-hub,rustls-tls
  • cargo check --all-targets --no-default-features --features hf-hub,native-tls
  • cargo fmt --all -- --check

The all-features OpenCV combination is covered by GitHub Actions; the local machine does not have pkg-config/OpenCV installed.

Signed-off-by: Bugen Zhao <i@bugenzhao.com>
Signed-off-by: Bugen Zhao <i@bugenzhao.com>
@BugenZhao
BugenZhao force-pushed the bz/align-smg-dependencies branch from 6e4715c to e003581 Compare July 14, 2026 04:43
@BugenZhao
BugenZhao merged commit 7df38e5 into main Jul 14, 2026
12 checks passed
@BugenZhao
BugenZhao deleted the bz/align-smg-dependencies branch July 14, 2026 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant