0003404: Background processes - Openbravo Issue Tracking System

Openbravo Issue Tracking System - Openbravo ERP

View Issue Details

Project

Category

View Status

Date Submitted

Last Update

0003404

Openbravo ERP

C. Security

public

2007-10-25 00:13

2009-05-20 07:21

Reporter

roklenardic

Assigned To

anthony_wolski

Priority

normal

Severity

minor

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

Fixed in Version

2.50

Merge Request Status

Review Assigned To

OBNetwork customer

Web browser

Modules

Core

Support ticket

Regression level

Regression date

Regression introduced in release

Regression introduced by commit

Triggers an Emergency Pack

Summary

0003404: Background processes

Description

In HttpSecureAppServlet.java, line 130 we have the following code

          try {
          object = new PeriodicBackground(this, Long.parseLong(strPeriodicBackgroundTime), strLogFileAcctServer, backgroundData[countBack].id, backgroundData[countBack].classname);
          } catch (IOException ioe) {
            ioe.printStackTrace();
          }

In case tomcat cannot write in the folder where the strLogFileAcctServer is put, all background processes fail to initialize/execute and noone ever knows about this. In case of Background accounting process, this is a huge issue.
instead of ioe.printStackTrace(); there should at least be log4j logging there.

On the other hand, there should be some kind of special logging/alerting for system critical processes such as background accounting so that the admin can see if they fail. maybe Alerts could be used for that. regardless, the exception catch there should be logged better.

Steps To Reproduce

Proposed Solution

Additional Information