[Yanel-dev] authentication ssl patch
Michael Wechner
michael.wechner at wyona.com
Wed Jan 16 12:32:20 CET 2008
Evaldas Taroza wrote:
> There was a small bug in YanelServlet. A check for ssl was needed.
done, thanks very much for your patch
Cheers
Michi
>------------------------------------------------------------------------
>
>Index: src/java/org/wyona/yanel/servlet/YanelServlet.java
>===================================================================
>--- src/java/org/wyona/yanel/servlet/YanelServlet.java (revision 30347)
>+++ src/java/org/wyona/yanel/servlet/YanelServlet.java (working copy)
>@@ -1151,7 +1151,11 @@
> } else {
> log.warn("Authentication was successful!");
> URL url = new URL(getRequestURLQS(request, null, false).toString());
>- url = new URL("https", url.getHost(), new Integer(sslPort).intValue(), url.getFile());
>+ if(sslPort != null){
>+ url = new URL("https", url.getHost(), new Integer(sslPort).intValue(), url.getFile());
>+ }else{
>+ // leave default
>+ }
> log.warn("Redirect to original request: " + url);
> response.setHeader("Location", url.toString());
> // TODO: Yulup has a bug re TEMPORARY_REDIRECT
>
>
>------------------------------------------------------------------------
>
>_______________________________________________
>Yanel-development mailing list
>Yanel-development at wyona.com
>http://lists.wyona.org/cgi-bin/mailman/listinfo/yanel-development
>
>
--
Michael Wechner
Wyona - Open Source Content Management - Yanel, Yulup
http://www.wyona.com
michael.wechner at wyona.com, michi at apache.org
+41 44 272 91 61
More information about the Yanel-development
mailing list