Merge pull request #63 from buehner/fix-auth-bug

looks good; fix needed in order to get JSON response by j_spring_security_check after logging in
terrestris · Jul 16, 2015 · 0221585 · 0221585
2 parents bb300bf + ee4c3b9
commit 0221585
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/main/java/de/terrestris/shogun/security/ShogunAuthProcessingFilter.java b/src/main/java/de/terrestris/shogun/security/ShogunAuthProcessingFilter.java
@@ -89,6 +89,9 @@ public void sendRedirect(HttpServletRequest httpServletRequest,
         });
         super.successfulAuthentication(request, response, authResult);
 
+        // set content type
+        response.setContentType("application/json;charset=UTF-8");
+
         // build a comma separated string of the ROLES
         String authorityText = StringUtils.join(authResult.getAuthorities(), ",");
 
@@ -109,8 +112,8 @@ public void sendRedirect(HttpServletRequest httpServletRequest,
         } catch (IOException e) {
             throw e;
         } finally {
-            IOUtils.closeQuietly(out);
             IOUtils.closeQuietly(jsonGenerator);
+            IOUtils.closeQuietly(out);
         }
     }