Page MenuHomePhabricator

Iterate over ranges correctly for encryped files
ClosedPublic

Authored by epriestley on Apr 5 2017, 1:35 PM.
Tags
None
Referenced Files
F18967233: D17623.diff
Fri, Nov 14, 8:38 PM
F18816963: D17623.diff
Tue, Oct 21, 10:57 AM
F18814596: D17623.id.diff
Mon, Oct 20, 11:34 PM
F18808997: D17623.diff
Oct 19 2025, 10:19 AM
F18780561: D17623.id42390.diff
Oct 12 2025, 5:24 PM
F18642615: D17623.id.diff
Sep 19 2025, 12:20 AM
F18636647: D17623.id42387.diff
Sep 17 2025, 4:52 AM
F18508707: D17623.id.diff
Sep 5 2025, 2:58 AM
Subscribers
None

Details

Summary

Fixes T12079. Currently, when a file is encrypted and a request has "Content-Range", we apply the range first, then decrypt the result. This doesn't work since you can't start decrypting something from somewhere in the middle (at least, not with our cipher selection).

Instead: decrypt the result, then apply the range.

Test Plan

Added failing unit tests, made them pass

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 5 2017, 4:35 PM
This revision was automatically updated to reflect the committed changes.