Project

General

Profile

Actions
Copy link

Bug #2812

closed

We are catching Throwable !

Added by François ARMAND over 11 years ago. Updated over 11 years ago.

Status:
Released
Priority:
2
Assignee:
François ARMAND
Category:
Architecture - Code maintenance
Target version:
2.4.0~beta5
Pull Request:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
Name check:
Fix check:
Regression:

Description

In several class in Rudder, we are catching Throwable ! We DO NOT have to catch that !

For example, we are catching OutOfMemory errors !

Change Throwable by Exception.

Liste of know cases:

- rudder/rudder-core/src/main/scala/com/normation/rudder/batch/AsyncDeploymentAgent.scala
- rudder/rudder-core/src/main/scala/com/normation/rudder/batch/UpdateDynamicGroups.scala
- rudder/rudder-core/src/main/scala/com/normation/rudder/repository/jdbc/EventLogJdbcRepository.scala

Actions
Copy link

Also available in: Atom PDF