Page MenuHomePhabricator

In ConduitCallFuture, only call Conduit exception messages on Conduit exceptions
ClosedPublic

Authored by epriestley on Sep 17 2020, 8:19 PM.
Tags
None
Referenced Files
F18903834: D21467.diff
Sat, Nov 8, 6:31 AM
F18872275: D21467.id.diff
Wed, Nov 5, 3:22 AM
F18856687: D21467.id51096.diff
Sat, Nov 1, 11:04 AM
F18821418: D21467.id51096.diff
Wed, Oct 22, 10:59 PM
F18821267: D21467.diff
Wed, Oct 22, 9:19 PM
F18788930: D21467.diff
Oct 15 2025, 7:08 AM
F18685021: D21467.id51097.diff
Sep 26 2025, 10:30 AM
F18682280: D21467.diff
Sep 26 2025, 5:37 AM
Subscribers
None

Details

Summary

Ref T13582. When this code is reached with a raw HTTP exception, it currently fatals.

Test Plan
  • Ran arc branches --conduit-uri=http://example.org (a bad Conduit URI).
  • Before: hard fatal with a bad method call.
  • After: non-Conduit exception raised to user. Not ideal, but a step forward.

Diff Detail

Repository
rARC Arcanist
Branch
ex1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 24911
Build 34368: Run Core Tests
Build 34367: arc lint + arc unit