Fix SLO issue

merge-requests/713/head
Vigneshan Seshamany 4 years ago
parent 84f3787fc5
commit 256d6c334f

@ -41,9 +41,9 @@ public class SsoLogoutHandler extends HttpServlet {
protected void doPost(HttpServletRequest req, HttpServletResponse resp) { protected void doPost(HttpServletRequest req, HttpServletResponse resp) {
for (String path : HandlerConstants.SSO_LOGOUT_COOKIE_PATHS) { for (String path : HandlerConstants.SSO_LOGOUT_COOKIE_PATHS) {
removeCookie(HandlerConstants.JSESSIONID_KEY, System.getProperty("iot.core.host"), path, resp); removeCookie(HandlerConstants.JSESSIONID_KEY, path, resp);
} }
removeCookie(HandlerConstants.COMMON_AUTH_ID_KEY, System.getProperty("iot.core.host"), "/", resp); removeCookie(HandlerConstants.COMMON_AUTH_ID_KEY, "/", resp);
ProxyResponse proxyResponse = new ProxyResponse(); ProxyResponse proxyResponse = new ProxyResponse();
proxyResponse.setCode(HttpStatus.SC_OK); proxyResponse.setCode(HttpStatus.SC_OK);
@ -58,11 +58,9 @@ public class SsoLogoutHandler extends HttpServlet {
} }
} }
private static void removeCookie(String cookieName, String domain, private static void removeCookie(String cookieName,String path, HttpServletResponse response) {
String path, HttpServletResponse response) {
Cookie cookie = new Cookie(cookieName, ""); Cookie cookie = new Cookie(cookieName, "");
cookie.setPath(path); cookie.setPath(path);
cookie.setDomain(domain);
cookie.setValue(null); cookie.setValue(null);
cookie.setMaxAge(0); cookie.setMaxAge(0);
response.addCookie(cookie); response.addCookie(cookie);

Loading…
Cancel
Save