> On 2012-04-12 12:14:33, Ryan Baxter wrote: > > LGTM Adam! If no one else has any feedback I will commit this tomorrow.
Committed revision 1326799 - Ryan ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4291/#review6874 ----------------------------------------------------------- On 2012-04-11 14:55:54, Adam Clarke wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4291/ > ----------------------------------------------------------- > > (Updated 2012-04-11 14:55:54) > > > Review request for shindig, Ryan Baxter and li xu. > > > Summary > ------- > > Allows multiple gadgets to share a token by binding to an OAuth2Client with > sharedToken=true > > > This addresses bug SHINDIG-1731. > https://issues.apache.org/jira/browse/SHINDIG-1731 > > > Diffs > ----- > > http://svn.apache.org/repos/asf/shindig/trunk/config/oauth2.json 1324775 > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/oauth2/oauth2_google_shared1.xml > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/oauth2/oauth2_google_shared2.xml > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/BasicOAuth2Store.java > 1324775 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/OAuth2Message.java > 1324775 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/persistence/OAuth2Client.java > 1324775 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/oauth2/persistence/sample/JSONOAuth2Persister.java > 1324775 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/oauth2/persistence/OAuth2ClientTest.java > 1324775 > > Diff: https://reviews.apache.org/r/4291/diff > > > Testing > ------- > > All test cases pass. > > > Thanks, > > Adam > >
