[PR #158] [CLOSED] limit chunks length on downloads #171

Closed
opened 2026-02-17 17:12:14 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Drop-OSS/drop-app/pull/158
Author: @laylafogiel-hash
Created: 10/12/2025
Status: Closed

Base: developHead: limit-chunk-length


📝 Commits (1)

  • 38f7483 limit chunks length on downloads

📊 Changes

1 file changed (+106 additions, -94 deletions)

View changed files

📝 src-tauri/src/games/downloads/download_logic.rs (+106 -94)

📄 Description

The drop server limits the length of requested chunks to 256 (see file server/api/v2/client/chunk.post.ts), but the client may request bigger chunks than that, which cuase an error. This especially can happen with .zip files which may have large chunks.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Drop-OSS/drop-app/pull/158 **Author:** [@laylafogiel-hash](https://github.com/laylafogiel-hash) **Created:** 10/12/2025 **Status:** ❌ Closed **Base:** `develop` ← **Head:** `limit-chunk-length` --- ### 📝 Commits (1) - [`38f7483`](https://github.com/Drop-OSS/drop-app/commit/38f7483d13efc4fc0ad1a5d3d48b58ff15481e61) limit chunks length on downloads ### 📊 Changes **1 file changed** (+106 additions, -94 deletions) <details> <summary>View changed files</summary> 📝 `src-tauri/src/games/downloads/download_logic.rs` (+106 -94) </details> ### 📄 Description The drop server limits the length of requested chunks to 256 (see file server/api/v2/client/chunk.post.ts), but the client may request bigger chunks than that, which cuase an error. This especially can happen with .zip files which may have large chunks. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-17 17:12:14 -05:00
yindo closed this issue 2026-02-17 17:12:14 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Drop-OSS/drop-app#171