Page MenuHomePhabricator

Stop the bleeding caused by attaching enormous patches to revision mail
ClosedPublic

Authored by epriestley on Sep 12 2017, 10:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 21, 1:54 PM
Unknown Object (File)
Thu, Dec 19, 12:49 AM
Unknown Object (File)
Wed, Dec 18, 3:51 AM
Unknown Object (File)
Sun, Dec 15, 10:13 AM
Unknown Object (File)
Mon, Dec 9, 4:22 PM
Unknown Object (File)
Mon, Dec 9, 4:22 PM
Unknown Object (File)
Mon, Dec 9, 4:22 PM
Unknown Object (File)
Mon, Dec 9, 4:21 PM
Subscribers
None

Details

Summary

Ref T12033. This is a very narrow fix for this issue, but it should fix the major error: don't attach patches if they're bigger than the mail body limit (by default, 512KB).

Specifically, the logs from an install in T12033 show a 112MB patch being attached, and that's the biggest practical problem here.

I'll follow up on the tasks with more nuanced future work.

Test Plan

Enabled differential.attach-patches, saw a patch attached to email. Set the byte limit very low, saw patches get thrown away.

Diff Detail

Repository
rP Phabricator
Branch
bytelimit1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18437
Build 24825: Run Core Tests
Build 24824: arc lint + arc unit