Senrian opened a new pull request, #1606:
URL: https://github.com/apache/struts/pull/1606

   ## Description
   Replace `e.printStackTrace()` with `System.err.println()` to properly log 
errors to stderr without stack trace noise in CLI tools.
   
   ## Changes
   - 
`plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/CompileReport.java`:
 Replace `e.printStackTrace()` with `System.err.println()` for cleaner error 
output
   
   ## Why
   Using `printStackTrace()` in production code is considered bad practice as 
it writes directly to stderr without proper logging framework integration. This 
change uses `System.err.println()` which is more appropriate for CLI tools.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to