Google Apps For Your Domain

10 views
Skip to first unread message

John

unread,
Dec 15, 2009, 4:27:00 PM12/15/09
to JOpenID Discussion Group
A patch to allow jopenid to work properly for GAFYD: just replace
OpenIdManager.getAuthenticationUrl(...) with:

public String getAuthenticationUrl(Endpoint endpoint, Association
association, String alias)
{
StringBuilder sb = new StringBuilder(1024);
sb.append(endpoint.getUrl());
if (endpoint.getUrl().contains("?"))
{
sb.append('&');
}
else
{
sb.append('?');
}
sb.append(getAuthQuery(alias)).append
("&openid.return_to=").append(returnToUrlEncode).append
("&openid.assoc_handle=").append(
association.getAssociationHandle());
if (realm != null)
sb.append("&openid.realm=").append(realm);
return sb.toString();
}


This is necessary to change
https://www.google.com/a/rentflow.net/o8/ud?be=o8?openid....
into
https://www.google.com/a/rentflow.net/o8/ud?be=o8&openid...

Note the first string has 2 question mark characters in it!

John

unread,
Dec 15, 2009, 4:42:56 PM12/15/09
to JOpenID Discussion Group
See also:
http://groups.google.com/group/google-federated-login-api/msg/9670a521a1dc8bb6

On Dec 15, 3:27 pm, John <john.tuni...@rentflow.net> wrote:
> A patch to allow jopenid to work properly for GAFYD:  just replace
> OpenIdManager.getAuthenticationUrl(...) with:
>
>    public String getAuthenticationUrl(Endpoint endpoint, Association
> association, String alias)
>     {
>         StringBuilder sb = new StringBuilder(1024);
>         sb.append(endpoint.getUrl());
>         if (endpoint.getUrl().contains("?"))
>         {
>             sb.append('&');
>         }
>         else
>         {
>             sb.append('?');
>         }
>         sb.append(getAuthQuery(alias)).append
> ("&openid.return_to=").append(returnToUrlEncode).append
> ("&openid.assoc_handle=").append(
>                 association.getAssociationHandle());
>         if (realm != null)
>             sb.append("&openid.realm=").append(realm);
>         return sb.toString();
>     }
>
> This is necessary to changehttps://www.google.com/a/rentflow.net/o8/ud?be=o8?openid....
> intohttps://www.google.com/a/rentflow.net/o8/ud?be=o8&openid...

Erwin Quinto

unread,
Dec 18, 2009, 7:46:21 AM12/18/09
to JOpenID Discussion Group
Hi John,

Thank you for this useful information. We will modify the
OpenIdManager as you suggested and include this modification on our
next release.

On Dec 16, 5:42 am, John <john.tuni...@rentflow.net> wrote:
> See also:http://groups.google.com/group/google-federated-login-api/msg/9670a52...

Reply all
Reply to author
Forward
0 new messages