Skip to content

Conversation

gaius-qi
Copy link
Member

@gaius-qi gaius-qi commented Aug 4, 2025

Description

This pull request includes updates to the versioning in Cargo.toml and an enhancement to the HTTP backend implementation to improve response handling. The most important changes are outlined below:

Version Updates in Cargo.toml:

  • Updated the workspace package version from 1.0.7 to 1.0.8 to reflect the new release. (Cargo.toml, Cargo.tomlL15-R15)
  • Updated dependencies (dragonfly-client and related crates) to version 1.0.8 for consistency with the new release. (Cargo.toml, Cargo.tomlL25-R31)

HTTP Backend Enhancement:

  • Added a Range header (bytes=0-) to HTTP requests in the dragonfly-client-backend to ensure the Content-Length is included in the response headers. This addresses scenarios where servers using Transfer-Encoding: chunked may omit Content-Length in HEAD requests. (dragonfly-client-backend/src/http.rs, dragonfly-client-backend/src/http.rsR141-R147)

Related Issue

Motivation and Context

Screenshots (if appropriate)

…se headers

Signed-off-by: Gaius <gaius.qi@gmail.com>
@gaius-qi gaius-qi added this to the v2.4.0 milestone Aug 4, 2025
@gaius-qi gaius-qi self-assigned this Aug 4, 2025
@gaius-qi gaius-qi added the enhancement New feature or request label Aug 4, 2025
Copy link
Member

@chlins chlins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Contributor

@LunaWhispers LunaWhispers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@chlins chlins enabled auto-merge (squash) August 4, 2025 06:18
@chlins chlins merged commit f0c9830 into main Aug 4, 2025
10 of 11 checks passed
@chlins chlins deleted the feature/transfer branch August 4, 2025 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants