How to handle cause by Exception in a good Java way

Question

I have some code in try-catch and in that code I am calling web service.On web service I have set timeout.

I have two web service call one taking no time its working fine and another one taking long to respond problem is not that but because of the time out it should throw SocketTimeoutException but its throwing PrivilegedActionException and after a long stack stace its showing cause by SocketTimeoutException.

I have intentionally put service call time very short to get SocketTimeoutException but its giving me PrivilegedActionException as a primary Exception.

I want to catch SocketTimeoutException and I'm not able to catch PrivilegedActionException becuase at the code level its showing PrivilegedActionException has not been thrown by this try catch.

I have written below code for achieve my goal but it's not working

try {
  //some code here for service call
}catch(SocketTimeoutException e)
{
  //not able to come here even though cause of the PrivilegedActionException is SocketTimeoutException 
}
catch(Exception e)
{
  //directly coming here OF COURSE
}

stacktrace:

java.security.PrivilegedActionException: com.sun.xml.internal.messaging.saaj.SOAPExceptionImpl: Message send failed
com.sun.xml.internal.messaging.saaj.SOAPExceptionImpl: java.security.PrivilegedActionException: com.sun.xml.internal.messaging.saaj.SOAPExceptionImpl: Message send failed

try `SocketTimeoutException | PrivilegedActionException` both in first catch block? Read more [Java find the first cause of an exception](http://stackoverflow.com/questions/1791610/java-find-the-first-cause-of-an-exception) — Braj, Mar 25 '15 at 04:04
Just for other viewers I am using java 1.6.....this is not supported.... — Abhishek Suthar, Mar 25 '15 at 05:25

Pham Trung · Accepted Answer · 2015-03-25T04:38:57.137

1

you can catch PrivilegedActionException, and call getCause to get SocketTimeoutException. Maybe a while loop is needed if SocketTimeoutException is not a direct cause.

try{

}catch(PrivilegedActionException e){
    Throwable tmp = e;
    while(tmp != null){
        if(tmp instanceof SocketTimeoutException){
            SocketTimeoutException cause = (SocketTimeoutException) tmp;
            //Do what you need to do here.
            break;
        }
        tmp = tmp.getCause();
    }
}

Temporary solution:

catch(Exception e){
    if(e instanceof PrivilegedActionException){
         //while loop here
    }
}

edited Mar 25 '15 at 04:38

answered Mar 25 '15 at 04:05

Pham Trung

11,204
2
24
43

thank for the answer but pham I have said that my compiler wont allow me to do catch PrivilegedActionException because the code which I have written in try catch its not throwing PrivilegedActionException – Abhishek Suthar Mar 25 '15 at 04:17
@AbhishekSuthar so just catch the correct exception, and apply the same logic to get the cause :) The basic idea is you need to call `getCause` to get what you want. – Pham Trung Mar 25 '15 at 04:19
you are saying that put this code in catch of Exception ? – Abhishek Suthar Mar 25 '15 at 04:21
@AbhishekSuthar that approach should work, just look ugly in code :( – Pham Trung Mar 25 '15 at 04:30
@AbhishekSuthar added a suggestion how you do it in `catch Exception` – Pham Trung Mar 25 '15 at 04:38

score 0 · Answer 2 · edited Mar 25 '15 at 04:59

0

Its better that you define your own Exception class and perform operations which are required.

Just make sure before throwing the exception you release all the resources which you have been using in your program.

edited Mar 25 '15 at 04:59

AbdulRahman Ansari

3,007
1
21
29

answered Mar 25 '15 at 04:17

user2760104

1
1

How to handle cause by Exception in a good Java way

2 Answers2