----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/1211/#review1926 -----------------------------------------------------------
Ship it! Committed revision 1171551. - Ryan On 2011-08-12 14:44:17, Dan Dumont wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1211/ > ----------------------------------------------------------- > > (Updated 2011-08-12 14:44:17) > > > Review request for shindig. > > > Summary > ------- > > While debugging using a container cross-site from where shindig is hosted, I > ran into trouble using: > config[osapi.container.ServiceConfig.API_PATH] = '/rpc'; > config[osapi.container.ServiceConfig.API_HOST] = // A url to a proxy on the > host server that points to the shindig server. > > I found that the osapi.container.Service.prototype.registerOsapiServices > function was defining methods that never ended up sticking, because other > methods would beat it to the punch. > This resulted in the above settings never taking hold. > > > This addresses bug SHINDIG-1569. > https://issues.apache.org/jira/browse/SHINDIG-1569 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/osapi.base/osapi.js > 1151610 > > Diff: https://reviews.apache.org/r/1211/diff > > > Testing > ------- > > Tested navigate gadget in a cross-hosted container environment (vanilla web > host loading CC from shindig server). > Tested navigate gadget in SampleContainer (same host). > > > Thanks, > > Dan > >