[Yanel-commits] rev 30831 - public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet

[michi at wyona.com]

Author: michi
Date: 2008-01-27 21:28:56 +0100 (Sun, 27 Jan 2008)
New Revision: 30831

Modified:
   public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
Log:
send 301 reponse and added some comments re 302

Modified: public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java
===================================================================
--- public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2008-01-27 19:42:49 UTC (rev 30830)
+++ public/yanel/trunk/src/webapp/src/java/org/wyona/yanel/servlet/YanelServlet.java	2008-01-27 20:28:56 UTC (rev 30831)
@@ -1159,10 +1159,13 @@
                     url = new URL("https", url.getHost(), new Integer(sslPort).intValue(), url.getFile());
                 }
                 log.warn("Redirect to original request: " + url);
+
+                //response.sendRedirect(url.toString()); // 302
+                // TODO: Yulup has a bug re TEMPORARY_REDIRECT (or is the problem that the load balancer is rewritting 302 reponses?!)
                 response.setHeader("Location", url.toString());
-                // TODO: Yulup has a bug re TEMPORARY_REDIRECT
-                //response.setStatus(javax.servlet.http.HttpServletResponse.SC_TEMPORARY_REDIRECT);
-                response.setStatus(javax.servlet.http.HttpServletResponse.SC_MOVED_PERMANENTLY);
+                response.setStatus(javax.servlet.http.HttpServletResponse.SC_MOVED_PERMANENTLY); // 301
+                //response.setStatus(javax.servlet.http.HttpServletResponse.SC_TEMPORARY_REDIRECT); // 302
+
                 return response;
             }
         } else {