Hi.
I'd like to have your feedback on 2 logging items related to jbossws-cxf and try-catch
[1].
1) Usage of Exception.printStackTrace() instead of logging feature
https://github.com/jbossws/jbossws-cxf/blob/master/modules/client/src/mai...
https://github.com/jbossws/jbossws-cxf/blob/master/modules/client/src/mai...
https://github.com/jbossws/jbossws-cxf/blob/master/modules/addons/transpo...
https://github.com/jbossws/jbossws-cxf/blob/master/modules/addons/transpo...
Is there any reason to call printStackTrace() directly ? If not I can create JIRA for
cleanup ++ check other jbossws-* projects.
2) // ignore in catch block
https://github.com/jbossws/jbossws-cxf/blob/master/modules/client/src/mai...
https://github.com/jbossws/jbossws-cxf/blob/master/modules/addons/transpo...
https://github.com/jbossws/jbossws-cxf/blob/master/modules/addons/transpo...
https://github.com/jbossws/jbossws-cxf/blob/master/modules/jaspi/src/main...
What about logging on trace level ? In case of UDPConduid it's catching exception on
socket.send() while looping across all network interfaces ... logging at trace could
potentially help when troubleshooting
What do you think about these items? Should I proceed with JIRA and other jbossws-*
projects?
Thank you.
Rostislav
[1] grep -b8 catch . -R | grep -v 'modules/test' | grep -v 'Test\.java' |
grep -v "\.xml" | uniq > ~/Downloads/jbossws-cxf-try-catch.txt