Skip to content

Share reqwest client for all file downloads, add timeout, use rustls, and allow compression#62

Merged
Gawdl3y merged 1 commit into
mainfrom
feat/shared-http
Dec 19, 2023
Merged

Share reqwest client for all file downloads, add timeout, use rustls, and allow compression#62
Gawdl3y merged 1 commit into
mainfrom
feat/shared-http

Conversation

@Gawdl3y

@Gawdl3y Gawdl3y commented Dec 19, 2023

Copy link
Copy Markdown
Owner

This does the following:

  • Shares a single reqwest HTTP client across all file downloads (manifest, mod artifacts, etc.)
  • Adds a 10s timeout to connections
  • Forces the use of rustls instead of native-tls (primarily so TLS 1.3 functions properly on Windows)
  • Allows response compression for requests

@Gawdl3y Gawdl3y added the enhancement Improvements/tweaks to existing features label Dec 19, 2023
@Gawdl3y Gawdl3y merged commit f850b86 into main Dec 19, 2023
@Gawdl3y Gawdl3y deleted the feat/shared-http branch December 19, 2023 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

enhancement Improvements/tweaks to existing features

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Gawdl3y