Proposed resolution for issue eclipse-ee4j#4649

Simple code changes to resolve eclipse-ee4j#4649
mmariotti · Oct 24, 2019 · 95f98f2 · 95f98f2
1 parent 05cd14d
commit 95f98f2
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/impl/src/main/java/com/sun/faces/context/ExternalContextImpl.java b/impl/src/main/java/com/sun/faces/context/ExternalContextImpl.java
@@ -713,7 +713,7 @@ public void redirect(String requestURI) throws IOException {
         doLastPhaseActions(ctx, true);
 
         if (ctx.getPartialViewContext().isPartialRequest()) {
-            if (getSession(true) instanceof HttpSession &&
+            if (response instanceof HttpServletResponse &&
                 ctx.getResponseComplete()) {
                 throw new IllegalStateException();
             }
@@ -731,8 +731,10 @@ public void redirect(String requestURI) throws IOException {
             pwriter.startDocument();
             pwriter.redirect(requestURI);
             pwriter.endDocument();
-        } else {
+        } else if (response instanceof HttpServletResponse) {
             ((HttpServletResponse) response).sendRedirect(requestURI);
+        } else {
+            throw new IllegalStateException();
         }
         ctx.responseComplete();