[ 
https://issues.apache.org/jira/browse/AXIS2-6086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17937371#comment-17937371
 ] 

Jeff Thomas commented on AXIS2-6086:
------------------------------------

I see you changed the log level to debug in [AXIS2-6065] and that ticket has 
version 2.0.0 but unless I am mistaken it is still error?

[https://github.com/apache/axis-axis2-java-core/blob/master/modules/transport/http/src/main/java/org/apache/axis2/transport/http/AxisServlet.java]

https://github.com/apache/axis-axis2-java-core/blob/v2.0.0/modules/transport/http/src/main/java/org/apache/axis2/transport/http/AxisServlet.java

> AxisServlet - processAxisFault - does not guard against NumberFormatException
> -----------------------------------------------------------------------------
>
>                 Key: AXIS2-6086
>                 URL: https://issues.apache.org/jira/browse/AXIS2-6086
>             Project: Axis2
>          Issue Type: Bug
>          Components: transports
>    Affects Versions: 2.0.0
>            Reporter: Jeff Thomas
>            Priority: Minor
>
> In the {{AxisServle#processAxisFault}} there is an unguarded 
> {{Integer.parseInt}} which could propagate a {{{}NumberFormatException{}}}.
> {code:java}
> if (status == null) {
>     log.error("processAxisFault() on error message: " + e.getMessage() + " , 
> found a null HTTP status from the MessageContext instance, setting 
> HttpServletResponse status to HttpServletResponse.SC_INTERNAL_SERVER_ERROR", 
> e);
>     res.setStatus(HttpServletResponse.SC_INTERNAL_SERVER_ERROR);
> } else {
>     log.error("processAxisFault() found an HTTP status from the 
> MessageContext instance, setting HttpServletResponse status to: " + status);
>     res.setStatus(Integer.parseInt(status));
>     return;
> } {code}
> ---
> Also these logs with level "error" are spammy - maybe consider a warning 
> since they are handled - one of our customers complained about thousands of 
> lines off errors with stacktraces for this recurring conditiion - our custom 
> fork has these set to warn.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscr...@axis.apache.org
For additional commands, e-mail: java-dev-h...@axis.apache.org

Reply via email to