----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3571/#review4529 -----------------------------------------------------------
Ship it! LGTM. Applied the patch and was able to get an oAuth2Spec section back in the ModulePrefs by adding "modulePrefs.oAuth2Spec" to the fields array on the metadata request. trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerApi.java <https://reviews.apache.org/r/3571/#comment10124> Small nit on whitespace here. - Stanton On 2012-01-20 19:47:21, Igor Belakovskiy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3571/ > ----------------------------------------------------------- > > (Updated 2012-01-20 19:47:21) > > > Review request for shindig and Henry Saputra. > > > Summary > ------- > > Need to expose OAuth2 metadata to the container, similar to > https://issues.apache.org/jira/browse/SHINDIG-1616 where we exposed Oauth 1.0 > data. > > > Diffs > ----- > > > trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerApi.java > 1232531 > > trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/GadgetsHandlerService.java > 1232531 > > Diff: https://reviews.apache.org/r/3571/diff > > > Testing > ------- > > Manual > > > Thanks, > > Igor > >
