1
0
Fork 0
forgejo/routers/api/actions
ThetaDev e80f8ff69f fix artifact range requests (#4218)
I noticed that Forgejo does not allow HTTP range requests when downloading artifacts. All other file downloads like releases and packages support them.

So I looked at the code and found that the artifact download endpoint uses a simple io.Copy to serve the file contents instead of using the established `ServeContentByReadSeeker` function which does take range requests into account.

Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4218
Reviewed-by: Earl Warren <earl-warren@noreply.codeberg.org>
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
Co-authored-by: ThetaDev <thetadev@magenta.de>
Co-committed-by: ThetaDev <thetadev@magenta.de>
2024-07-10 05:28:01 +00:00
..
ping [CHORE] Update connect-go to maintained fork 2024-02-28 09:40:56 +01:00
runner Remove deprecated stuff for runners (#30930) 2024-05-12 20:03:10 +02:00
actions.go Refactor web package and context package (#25298) 2023-06-18 09:59:09 +02:00
artifact.pb.go Actions Artifacts v4 backend (#28965) 2024-03-06 12:10:45 +08:00
artifact.proto Actions Artifacts v4 backend (#28965) 2024-03-06 12:10:45 +08:00
artifacts.go Enable unparam linter (#31277) 2024-06-16 13:42:58 +02:00
artifacts_chunks.go Reduce memory usage for chunked artifact uploads to MinIO (#31325) 2024-06-16 13:42:58 +02:00
artifacts_utils.go Enable unparam linter (#31277) 2024-06-16 13:42:58 +02:00
artifactsv4.go fix artifact range requests (#4218) 2024-07-10 05:28:01 +00:00