[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