Page MenuHomePhabricator

Policy - lock down file loading in mail reply handler path
ClosedPublic

Authored by btrahan on Jan 30 2015, 11:14 PM.
Tags
None
Referenced Files
F15451876: D11589.id.diff
Sat, Mar 29, 12:15 AM
F15446343: D11589.diff
Thu, Mar 27, 6:34 PM
F15445705: D11589.id.diff
Thu, Mar 27, 3:35 PM
F15437838: D11589.id27883.diff
Tue, Mar 25, 9:45 PM
F15432955: D11589.id27991.diff
Mon, Mar 24, 8:32 PM
Unknown Object (File)
Feb 8 2025, 2:09 AM
Unknown Object (File)
Feb 8 2025, 2:09 AM
Unknown Object (File)
Feb 8 2025, 2:09 AM
Subscribers

Details

Summary

Ref T7094. This one is really straight-forward since $this->actor is always populated and the right thing to do here.

Test Plan

used the ole thinking noodle since testing email w/ attachments is really hard

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Policy - lock down file loading in mail reply handler path.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Jan 31 2015, 2:26 PM
This revision was automatically updated to reflect the committed changes.