Opened at 2020-09-17T20:59:54Z
Last modified at 2023-09-05T14:20:22Z
#3420 assigned defect
Twisted web Resources should "return ErrorPage" instead of "raise WebError" — at Version 3
| Reported by: | sajith | Owned by: | sajith |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | code-frontend-web | Version: | n/a |
| Keywords: | Cc: | ||
| Launchpad Bug: |
Description (last modified by exarkun)
In case of errors, twisted.web Resource instances should be responding with an error page, instead of throwing an exception. Any raise WebError instances we have in Tahoe WUI should be replaced with return ErrorPage.
More background is in this discussion at GitHub about a PR that solves #3382.
Sub-tickets:
- https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3422
- https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3423
- https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3424
- https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3425
- https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3426
- https://tahoe-lafs.org/trac/tahoe-lafs/ticket/3427
Change History (3)
comment:1 Changed at 2020-09-17T21:00:21Z by sajith
- Component changed from unknown to code-frontend-web
- Milestone changed from undecided to Support Python 3
- Owner set to sajith
- Status changed from new to assigned
comment:2 Changed at 2020-09-18T17:59:01Z by exarkun
comment:3 Changed at 2020-09-18T18:02:21Z by exarkun
- Description modified (diff)
Note: See
TracTickets for help on using
tickets.
This is probably worth splitting up. The changes will be spread out across a lot of the web code and have little overlap (apart from a few shared error page helpers that will come pretty early, I suspect).