Pass exception to PhutilProxyException
ClosedPublic

Authored by jcox on Feb 10 2017, 5:30 PM.

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
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
jcox created this revision.Feb 10 2017, 5:30 PM
jcox updated this revision to Diff 41691.Feb 10 2017, 5:32 PM

learn to count parens

This revision is now accepted and ready to land.Feb 10 2017, 5:35 PM
This revision was automatically updated to reflect the committed changes.