Page MenuHomePhabricator

Pass exception to PhutilProxyException
ClosedPublic

Authored by jcox on Feb 10 2017, 5:30 PM.
Tags
None
Referenced Files
F19813653: D17335.diff
Wed, Mar 4, 10:17 AM
F19697085: D17335.diff
Tue, Feb 10, 3:30 PM
F19605216: D17335.diff
Tue, Feb 3, 8:55 AM
F19533791: D17335.id41691.diff
Jan 20 2026, 8:21 PM
F19505975: D17335.id41691.diff
Jan 9 2026, 3:48 PM
F19152180: D17335.id41692.diff
Dec 11 2025, 1:11 AM
F18873382: D17335.id41690.diff
Nov 5 2025, 9:26 AM
F18779751: D17335.id41691.diff
Oct 11 2025, 3:19 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
Branch
PassExceptionToProxyMethod (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15588
Build 20569: Run Core Tests
Build 20568: arc lint + arc unit