Page MenuHomePhabricator

Pass exception to PhutilProxyException
ClosedPublic

Authored by jcox on Feb 10 2017, 5:30 PM.
Tags
None
Referenced Files
F15476581: D17335.id.diff
Mon, Apr 7, 7:30 AM
F15475174: D17335.diff
Sun, Apr 6, 4:56 PM
F15467248: D17335.id41692.diff
Thu, Apr 3, 1:51 PM
F15444180: D17335.diff
Thu, Mar 27, 8:18 AM
F15443387: D17335.id41692.diff
Thu, Mar 27, 4:09 AM
F15424597: D17335.diff
Sat, Mar 22, 11:14 PM
F15420202: D17335.id41692.diff
Fri, Mar 21, 12:10 PM
F15402517: D17335.diff
Mon, Mar 17, 10:53 PM
Subscribers

Details

Summary

Fixes T12243. That error occured due to network flakiness with some mounted filesystems so I'm not sure how best to simulate it. But you can look and see that the PhutilProxyException does indeed expect an exception as its second arg.

Test Plan

Look at method signature... look at callsite... now back at the method. Smile and nod.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable