Page MenuHomePhabricator

Replace AphrontUsageException with AphrontMalformedRequestException
ClosedPublic

Authored by epriestley on Sep 3 2015, 12:33 PM.
Tags
None
Referenced Files
F14017786: D14047.diff
Mon, Nov 4, 10:54 PM
F14006301: D14047.diff
Mon, Oct 28, 5:47 AM
F13980143: D14047.id.diff
Oct 19 2024, 8:42 AM
F13969635: D14047.id33955.diff
Oct 17 2024, 3:29 AM
F13964610: D14047.id33956.diff
Oct 15 2024, 9:09 PM
F13964609: D14047.id33955.diff
Oct 15 2024, 9:09 PM
F13964608: D14047.id.diff
Oct 15 2024, 9:09 PM
F13964607: D14047.diff
Oct 15 2024, 9:09 PM
Subscribers
None
Tokens
"Like" token, awarded by demo.

Details

Summary

Ref T1806. Ref T7173. Context here is that I want to fix "you can not log in to this instance" being a confusing mess with an opaque error. To do this without hacks, I want to:

  • clean up some exception handling behavior (this diff);
  • modularize exception handling (next diff);
  • replace confusing, over-general exceptions with tailored ones in the Phacility cluster, using the new modular stuff.

This cleans up an awkward "AphrontUsageException" which does some weird stuff right now. In particular, it is extensible and extended in one place in Diffusion, but that extension is meaningless.

Realign this as "AphrontMalformedRequestException", which is a better description of what it is and does: raises errors before we can get as far as normal routing and site handling.

Test Plan

Hit some of these exceptions, saw the expected "abandon all hope" error page.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Replace AphrontUsageException with AphrontMalformedRequestException.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
This revision is now accepted and ready to land.Sep 3 2015, 12:38 PM
This revision now requires review to proceed.Sep 3 2015, 12:39 PM
  • Improve logging behavior and describe things in more detail.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 3 2015, 3:39 PM
This revision was automatically updated to reflect the committed changes.