Project

General

Profile

Actions

Improvements #10137

closed
RS RS

Uncaught exceptions in handleRequest crash the application

Improvements #10137: Uncaught exceptions in handleRequest crash the application

Added by Roel Standaert over 4 years ago. Updated about 4 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.

RS Updated by Roel Standaert over 4 years ago Actions #1

  • Description updated (diff)

RS Updated by Roel Standaert about 4 years ago Actions #2

  • Status changed from New to InProgress
  • Assignee set to Roel Standaert

RS Updated by Roel Standaert about 4 years ago Actions #3

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

RS Updated by Roel Standaert about 4 years ago Actions #4

  • Status changed from Review to Implemented @Emweb

RS Updated by Roel Standaert about 4 years ago Actions #5

  • % Done changed from 0 to 100

RS Updated by Roel Standaert about 4 years ago Actions #6

  • Status changed from Implemented @Emweb to Resolved

RS Updated by Roel Standaert about 4 years ago Actions #7

  • Assignee set to Roel Standaert

RS Updated by Roel Standaert about 4 years ago Actions #8

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom