----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1067/#review1024 -----------------------------------------------------------
Ship it! I always like removing excess code wherever possible. - Paul On 2011-07-11 07:02:12, Henry Saputra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1067/ > ----------------------------------------------------------- > > (Updated 2011-07-11 07:02:12) > > > Review request for shindig. > > > Summary > ------- > > Remove the TestSecurityTokenCodec for common container to make the OAuth > gadget works by allowing DefaultSecurityTokenCodec to create the right st. > > > Diffs > ----- > > trunk/content/samplecontainer/examples/commoncontainer/assembler.js 1145041 > > trunk/java/server/src/main/java/org/apache/shindig/sample/commoncontainer/auth/CommonContainerAuthGuiceModule.java > 1145041 > > trunk/java/server/src/main/java/org/apache/shindig/sample/commoncontainer/auth/CommonContainerSecurityTokenCodec.java > 1145041 > > trunk/java/server/src/main/java/org/apache/shindig/sample/commoncontainer/auth/TestSecurityTokenCodec.java > 1145041 > trunk/java/server/src/main/webapp/WEB-INF/web.xml 1145041 > > Diff: https://reviews.apache.org/r/1067/diff > > > Testing > ------- > > pass unit test and run gadgets in common container sample page. > > > Thanks, > > Henry > >
