search

teintinu / dyuproject

Automatically exported from code.google.com/p/dyuproject
Apache License 2.0
1 stars 0 forks source link

Yes, I do need openIdServer after the user is authenticated #24

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 

After authentication of a google apps user, I use the openIdServer to detect 
they came from google and skip email verification for those users.

BUT, in com.dyuproject.openid.OpenIdUser.setIdentity(String) it has:

{{{
            // TODO confirm that nobody needs this when user is already authenticated
            _openIdServer = null;
            _openIdDelegate = null;
}}}

So I have to specially preserve the _openIdServer myself to be able to detect 
this after the auth is complete.

Be nice if these lines were omitted.

Original issue reported on code.google.com by dob...@gmail.com on 10 Sep 2010 at 6:38