Merge branch '1.5.x'
This commit is contained in:
@@ -177,7 +177,7 @@ public class ErrorPageFilter implements Filter, ErrorPageRegistry {
|
||||
}
|
||||
setErrorAttributes(request, 500, ex.getMessage());
|
||||
request.setAttribute(ERROR_EXCEPTION, ex);
|
||||
request.setAttribute(ERROR_EXCEPTION_TYPE, ex.getClass().getName());
|
||||
request.setAttribute(ERROR_EXCEPTION_TYPE, ex.getClass());
|
||||
response.reset();
|
||||
response.sendError(500, ex.getMessage());
|
||||
request.getRequestDispatcher(path).forward(request, response);
|
||||
|
||||
@@ -262,7 +262,7 @@ public class ErrorPageFilterTests {
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_MESSAGE))
|
||||
.isEqualTo("BAD");
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_EXCEPTION_TYPE))
|
||||
.isEqualTo(RuntimeException.class.getName());
|
||||
.isEqualTo(RuntimeException.class);
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_REQUEST_URI))
|
||||
.isEqualTo("/test/path");
|
||||
assertThat(this.response.isCommitted()).isTrue();
|
||||
@@ -319,7 +319,7 @@ public class ErrorPageFilterTests {
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_MESSAGE))
|
||||
.isEqualTo("BAD");
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_EXCEPTION_TYPE))
|
||||
.isEqualTo(IllegalStateException.class.getName());
|
||||
.isEqualTo(IllegalStateException.class);
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_REQUEST_URI))
|
||||
.isEqualTo("/test/path");
|
||||
assertThat(this.response.isCommitted()).isTrue();
|
||||
@@ -493,7 +493,7 @@ public class ErrorPageFilterTests {
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_MESSAGE))
|
||||
.isEqualTo("BAD");
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_EXCEPTION_TYPE))
|
||||
.isEqualTo(RuntimeException.class.getName());
|
||||
.isEqualTo(RuntimeException.class);
|
||||
assertThat(this.request.getAttribute(RequestDispatcher.ERROR_REQUEST_URI))
|
||||
.isEqualTo("/test/path");
|
||||
assertThat(this.response.isCommitted()).isTrue();
|
||||
|
||||
Reference in New Issue
Block a user