Page MenuHomePhabricator

Gracefully fail request if non existing callsign is passed to getrecentcommitsbypath instead of crashing
ClosedPublic

Authored by artms on Aug 2 2018, 2:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 3 2024, 7:11 PM
Unknown Object (File)
Jan 11 2024, 12:12 PM
Unknown Object (File)
Jan 11 2024, 12:11 PM
Unknown Object (File)
Jan 5 2024, 1:35 PM
Unknown Object (File)
Dec 28 2023, 10:55 PM
Unknown Object (File)
Dec 27 2023, 10:39 PM
Unknown Object (File)
Dec 26 2023, 11:40 AM
Unknown Object (File)
Dec 22 2023, 11:44 PM
Subscribers

Details

Summary

diffusion.getrecentcommitsbypath fails with 500 error when non existing callsign is passed:

>>> UNRECOVERABLE FATAL ERROR <<<

Call to a member function getCommit() on null

Expected Behavior:
Return more graceful error notifying caller that such callsign/repository does not exist

Reproduction steps:
Open conduit: https://secure.phabricator.com/conduit/method/diffusion.getrecentcommitsbypath/
Enter:
callsign: "obviouslynotexisting"
path: "/random"
Click call method

Test Plan

after applying patch - call no longer fails with 500s

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

artms requested review of this revision.Aug 2 2018, 2:46 PM

This doesn't have any callers and should probably be frozen/deprecated, but this change won't hurt anything.

This revision is now accepted and ready to land.Aug 2 2018, 2:50 PM

Uh, we had couple of cases and this caused extra 500s in logs...

This revision was automatically updated to reflect the committed changes.