loris-imageserver / loris

Loris IIIF Image Server
Other
209 stars 87 forks source link

Refactor the Exception classes to drop the notion of "http_status" #384

Closed alexwlchan closed 6 years ago

alexwlchan commented 6 years ago

The value of this attribute is never read except in tests, and setting it on internal exceptions is arguably wrong – it’s a presentation aspect that affects the HTTP response to the user, so should really only be set in webapp.py (which is where all the values we actually use are defined!).