Skip to content

Conversation

@francoo98
Copy link

@francoo98 francoo98 commented Jan 2, 2026

This pr is a draft to fix #709.
Atm, it only supports one range in the Range header. Example: Range: bytes=0-5000
/claim #709

@netlify
Copy link

netlify bot commented Jan 2, 2026

Deploy Preview for zio-http ready!

Name Link
🔨 Latest commit 8e0a246
🔍 Latest deploy log https://app.netlify.com/projects/zio-http/deploys/695d7e6359595d0008587a73
😎 Deploy Preview https://deploy-preview-3877--zio-http.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@CLAassistant
Copy link

CLAassistant commented Jan 2, 2026

CLA assistant check
All committers have signed the CLA.

@francoo98 francoo98 changed the title Implementing support for HTTP Range header (#709) WIP: Implementing support for HTTP Range header (#709) Jan 3, 2026
@francoo98
Copy link
Author

Do you have any thoughts on fixing the MIMA check? I changed the definition of fromFile in Body.scala.
Current:
fromFile(file: java.io.File, chunkSize: Int = 1024 * 4)(implicit trace: Trace): ZIO[Any, Nothing, Body]
This pr:
fromFile(file: java.io.File, chunkSize: Int = 1024 * 4, start: Long = 0, end: Option[Long] = None)(implicit trace: Trace): ZIO[Any, Nothing, Body]
Should I modify the definition or create a new method fromRangedFile?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support Http Range header on request for Files

2 participants