LibreKAT/Ocideck
LibreKAT/Ocideck
Watch
4
0
Fork
You've already forked Ocideck
 0
Code Issues 2 Pull requests Projects Releases 4 Packages Activity Actions

bug(package): maak exportbudget consistent met de herimportgrens en stream assets #1046

New issue
Closed
opened 2026-08-01 08:26:06 +00:00 by brenno · 1 comment
brenno commented 2026-08-01 08:26:06 +00:00
Owner
Copy link

Context
Package-export leest elke asset volledig in een Archive en materialiseert daarna ook de volledige ZIP. Er is geen cumulatief exportbudget. Eén toegestaan videobestand van 1 GiB kan zo een pakket maken dat OciDeck zelf weigert te heropenen vanwege de 512 MiB importgrens.

Impact
Een export kan meerdere gigabytes geheugen vragen en een succesvol geschreven maar niet-herimporteerbaar .ocideck-bestand opleveren.

Relevante code

  • lib/services/file/file_service_package.dart:53 en 110
  • lib/services/file_service.dart:753

Acceptatie

  • Stat assets vóór lezen en dwing een cumulatief pakketbudget af dat aansluit op de importgrens.
  • Een succesvol geëxporteerd pakket is door dezelfde versie herimporteerbaar.
  • Gebruik streaming ZIP-uitvoer of een ander begrensd pad zodat bronassets en volledige ZIP niet allemaal tegelijk in geheugen staan.
  • Tests bewaken cumulatieve grens, grote losse asset en export-herimport-contract.
Context Package-export leest elke asset volledig in een Archive en materialiseert daarna ook de volledige ZIP. Er is geen cumulatief exportbudget. Eén toegestaan videobestand van 1 GiB kan zo een pakket maken dat OciDeck zelf weigert te heropenen vanwege de 512 MiB importgrens. Impact Een export kan meerdere gigabytes geheugen vragen en een succesvol geschreven maar niet-herimporteerbaar .ocideck-bestand opleveren. Relevante code - lib/services/file/file_service_package.dart:53 en 110 - lib/services/file_service.dart:753 Acceptatie - Stat assets vóór lezen en dwing een cumulatief pakketbudget af dat aansluit op de importgrens. - Een succesvol geëxporteerd pakket is door dezelfde versie herimporteerbaar. - Gebruik streaming ZIP-uitvoer of een ander begrensd pad zodat bronassets en volledige ZIP niet allemaal tegelijk in geheugen staan. - Tests bewaken cumulatieve grens, grote losse asset en export-herimport-contract.
brenno commented 2026-08-01 19:49:18 +00:00
Author
Owner
Copy link

Opgelost op main (PR #1089, merge in 03cd9dd5). Pakket-export stat't nu elke asset vóór het lezen en dwingt een cumulatief budget af dat aansluit op de 512 MiB-importgrens (PackageBudgetExceeded, met melding in 31 talen); een geslaagd pakket is door dezelfde versie herimporteerbaar. Tests: grote losse asset, cumulatieve grens op de byte, export-herimport-contract.

Opgelost op main (PR #1089, merge in 03cd9dd5). Pakket-export stat't nu elke asset vóór het lezen en dwingt een cumulatief budget af dat aansluit op de 512 MiB-importgrens (PackageBudgetExceeded, met melding in 31 talen); een geslaagd pakket is door dezelfde versie herimporteerbaar. Tests: grote losse asset, cumulatieve grens op de byte, export-herimport-contract.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/markdown-editor-gate-followup-1123
feat/native-calls-f2-xmpp
codex/markdown-editor-translations
codex/markdown-editor-ux
codex/template-followup
cockpit-wip
v0.1.3
v0.1.2
v0.1.1
v0.1.0
archive/git-mirror
Labels
Clear labels   
accepted
Assessed and will be built

  
bug
Something is broken

  
declined
Assessed and will not be built

  
docs
Documentation only

  
duplicate
Already tracked elsewhere

  
enhancement
New feature or improvement request

  
good first issue
Small, self-contained, and a good place to start

  
in-progress
Someone is working on this right now. Do not start it in parallel.

  
needs-info
Waiting for the reporter

  
privacy
Personal data can leak. Handled before anything else.

  
security
A weakness someone could use. Handled before anything else.

  
triage
Awaiting assessment

No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
LibreKAT/Ocideck#1046
No description provided.