Page MenuHomePhabricator

Fix file encoding migration
ClosedPublic

Authored by kaendfinger2 on Jul 22 2018, 4:50 AM.
Tags
Referenced Files
Unknown Object (File)
Mon, Mar 4, 10:59 PM
Unknown Object (File)
Feb 22 2024, 3:04 PM
Unknown Object (File)
Feb 15 2024, 11:02 PM
Unknown Object (File)
Feb 3 2024, 7:07 PM
Unknown Object (File)
Jan 27 2024, 3:27 AM
Unknown Object (File)
Jan 25 2024, 1:14 AM
Unknown Object (File)
Jan 9 2024, 10:52 PM
Unknown Object (File)
Jan 5 2024, 1:36 PM
Subscribers

Details

Summary

See Discourse

When migrating to aes-256-cbc, integrity hashes were not updated, so data was not properly

Test Plan

I ran this test script to ensure it worked.
I created some files with lipsum, ensured that after encoding them with aes-256-cbc, they were not able to be cat'd.
After applying this patch and rerunning the script, it worked successfully.

Diff Detail

Repository
rP Phabricator
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

Note that the lints and unit tests do pass on my end. I used the "Create Diff" action to do this instead of arc diff (which was silly of me)

Thanks! I was able to follow your steps to reproduce this and verify the fix.

The integrity hash code was added after the original encryption code and this particular pathway must have escaped testing.

This revision is now accepted and ready to land.Jul 23 2018, 10:13 PM
This revision was automatically updated to reflect the committed changes.