Rethrow wrapped ConcurrencyExceptions. Also updated code to
rethrow wrapped TimeoutExceptions.
This commit is contained in:
parent
846ff46e1e
commit
6da249a73d
1 changed files with 11 additions and 12 deletions
|
@ -276,26 +276,25 @@ public class RhinoEngine implements ScriptingEngine {
|
||||||
} catch (ConcurrencyException concur) {
|
} catch (ConcurrencyException concur) {
|
||||||
throw concur;
|
throw concur;
|
||||||
} catch (Exception x) {
|
} catch (Exception x) {
|
||||||
// check if this is a redirect exception, which has been converted by fesi
|
|
||||||
// into an EcmaScript exception, which is why we can't explicitly catch it
|
|
||||||
if (reval.res.getRedirect() != null) {
|
|
||||||
throw new RedirectException(reval.res.getRedirect());
|
|
||||||
}
|
|
||||||
|
|
||||||
// do the same for not-modified responses
|
|
||||||
if (reval.res.getNotModified()) {
|
|
||||||
throw new RedirectException(null);
|
|
||||||
}
|
|
||||||
|
|
||||||
// has the request timed out? If so, throw TimeoutException
|
// has the request timed out? If so, throw TimeoutException
|
||||||
if (thread != Thread.currentThread())
|
if (thread != Thread.currentThread())
|
||||||
throw new TimeoutException ();
|
throw new TimeoutException ();
|
||||||
|
|
||||||
// create and throw a ScriptingException with the right message
|
// create and throw a ScriptingException with the right message
|
||||||
String msg;
|
String msg;
|
||||||
if (x instanceof JavaScriptException) {
|
if (x instanceof JavaScriptException) {
|
||||||
msg = ((JavaScriptException) x).getValue().toString();
|
msg = ((JavaScriptException) x).getValue().toString();
|
||||||
} else if (x instanceof WrappedException) {
|
} else if (x instanceof WrappedException) {
|
||||||
msg = ((WrappedException) x).getWrappedException().toString();
|
Throwable wrapped = ((WrappedException) x).getWrappedException();
|
||||||
|
// if this is a wrapped concurrencyException, rethrow it.
|
||||||
|
if (wrapped instanceof ConcurrencyException) {
|
||||||
|
throw (ConcurrencyException) wrapped;
|
||||||
|
}
|
||||||
|
// also check if this is a wrapped redirect exception
|
||||||
|
if (wrapped instanceof RedirectException) {
|
||||||
|
throw (RedirectException) wrapped;
|
||||||
|
}
|
||||||
|
msg = wrapped.toString();
|
||||||
} else {
|
} else {
|
||||||
msg = x.toString();
|
msg = x.toString();
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Reference in a new issue