Li, thanks for the response.

Ya, it's just unfortunate I have to overwrite the entire class.  I tried to
extend the class and overwrite the method, but since it's static that isn't
going to work (guice complains about it already being bound).  I noticed on
this link

http://docs.opensocial.org/display/OSD/Client+Authentication

There is a comment

TODO : Should this be a list?  Like
"client_authentication" : "Basic, MyAuthExtension"

I wonder if there was thought that this should be in a configuration file
and not done via code?  Not sure.  At any rate, I'll just go with the
overriding method and cross my fingers this class doesn't change very often.
We have a few other guice modules that we've overriden and it's a pain to
keep them in synch.

doug

On 12/6/11 3:55 PM, "Li Xu" <[email protected]> wrote:

> Hi, Doug
> To my understanding, yes,  it need to overwrite OAuth2HandlerModule to
> provide the new AuthenticationHandler list.
> Thanks for the catch about typo, will fix it.
> thanks,
> li


Reply via email to