> On 2012-05-22 12:20:21, Stanton Sievers wrote: > > While the sample gadget that you added does use the new APIs it is not > > obvious from the interaction with the gadget that they are being used. Are > > the images on picasa themselves not protected? Only the feed is? A > > compelling demonstration would be to insert img tags into the DOM of the > > gadget that displays a handful of pictures from picasa via the proxy. This > > obviously doesn't work as a proper demo of the APIs if the images aren't > > protected. > > > > Can you update the text in the gadget to indicate that one needs a picasa > > account before they can use this gadget? > > > > It would also be great to have this gadget listed in the sample common > > container's drop-down list of gadgets. You can add it in > > shindig-project/content/samplecontainer/examples/commoncontainer/gadgetCollections.json
In Picasa API, only feeds are protected. Frankly, I don't like to add image service into Shindig just for an example. I've tries Spring OAuth2 sample, and it works well. Do you think it is OK to use Spring OAuth2 sample provider? I will update the text in the gadget to include the instructions on how to setup the environment and add the sample gadget to gadgetCollections.json once I get the sample gadget ready. - Xiao Feng ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5112/#review8029 ----------------------------------------------------------- On 2012-05-22 08:51:49, Xiao Feng Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5112/ > ----------------------------------------------------------- > > (Updated 2012-05-22 08:51:49) > > > Review request for shindig, Ryan Baxter, Dan Dumont, Stanton Sievers, and > Brian Lillie. > > > Summary > ------- > > Couple of changes are included in this patch. > 1) On the client side, the getProxyUrl is updated to add auth parameter to > specify the auth scheme used, also check for the AUTHORIZATION and > OAUTH_SERVICE setting and add them in proxy url. > 2) On the server side, proxy servlet will pass additional HttpServletRequest > to ProxyHandler to build the HttpRequest object, in the ProxyHandler > additional information as security token, auth type, oauth service and gadget > will be used to construct a HttpRequest to pass to the DefaultRequestPipeline > for handling. > > > This addresses bug Shindig-1773. > https://issues.apache.org/jira/browse/Shindig-1773 > > > Diffs > ----- > > http://svn.apache.org/repos/asf/shindig/trunk/config/container.js 1341083 > http://svn.apache.org/repos/asf/shindig/trunk/config/oauth2.json 1340019 > > http://svn.apache.org/repos/asf/shindig/trunk/content/samplecontainer/examples/oauth2/oauth2_google_proxy.xml > PRE-CREATION > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/core.io/io.js > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/test/javascript/features/core.io/iotest.js > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/servlet/ProxyServlet.java > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/ProxyUriBase.java > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/ProxyUriManager.java > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/UriCommon.java > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ProxyHandlerTest.java > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/gadgets/src/test/java/org/apache/shindig/gadgets/servlet/ProxyServletTest.java > 1341083 > > http://svn.apache.org/repos/asf/shindig/trunk/java/server-resources/src/main/webapp/WEB-INF/web.xml > 1341083 > > Diff: https://reviews.apache.org/r/5112/diff > > > Testing > ------- > > > Thanks, > > Xiao Feng > >
