Project

General

Profile

Actions

Improvements #10137

closed

Uncaught exceptions in handleRequest crash the application

Added by Roel Standaert over 2 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Roel Standaert
Target version:
Start date:
04/08/2022
Due date:
% Done:

100%

Estimated time:

Description

In the past I wasn't quite so sure about whether or not we should catch all exceptions in WResource::handleRequest by default.

Arguments for catching exceptions:

  • programming errors don't crash the entire application, which is probably preferred in production

Arguments for not catching:

  • not catching makes developers using Wt notice the error, and allow them to debug and fix them. Catching and logging is less obvious than crashing.

I think we should catch by default, and perhaps allow developers to turn it off to facilitate debugging.

Actions #1

Updated by Roel Standaert over 2 years ago

  • Description updated (diff)
Actions #2

Updated by Roel Standaert over 2 years ago

  • Status changed from New to InProgress
  • Assignee set to Roel Standaert
Actions #3

Updated by Roel Standaert over 2 years ago

  • Status changed from InProgress to Review
  • Assignee deleted (Roel Standaert)
Actions #4

Updated by Roel Standaert over 2 years ago

  • Status changed from Review to Implemented @Emweb
Actions #5

Updated by Roel Standaert over 2 years ago

  • % Done changed from 0 to 100
Actions #6

Updated by Roel Standaert over 2 years ago

  • Status changed from Implemented @Emweb to Resolved
Actions #7

Updated by Roel Standaert over 2 years ago

  • Assignee set to Roel Standaert
Actions #8

Updated by Roel Standaert over 2 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF