On Fri, Mar 7, 2008 at 6:27 AM, Cassie <[EMAIL PROTECTED]> wrote: > Arne - When I change the syndicator to use > /gadgets/files/container/rpc_relay.html as the url, the request goes > through > but then I get an "Invalid auth token" javascript error when calling > something like adjustHeight(). > > What else did you need to do to get it to work in the samplecontainer? > > Or, does anybody know why an invalid auth token error would occur with > this > change?
It shouldn't -- auth token validation is based on the iframe id, not the url. I'll take a look. > > I cleaned up the samplecontainer html so that the gadget iframe isn't > 100%.. > which means I really want dynamic-height to work now :) > > This should be reproducable from head with the one char syndicator.js > change. > > - Cassie > > > On Fri, Mar 7, 2008 at 12:33 PM, Kevin Brown <[EMAIL PROTECTED]> wrote: > > > The default synd should be set up specifically for the sample files -- > > period. If someone is using a different default path (such as a > production > > site), they should provide their own syndicator config. > > > > On Fri, Mar 7, 2008 at 3:01 AM, Cassie <[EMAIL PROTECTED]> wrote: > > > > > Can't we just add some code to rpc.js which will append a slash if the > > > parent does not end in a slash? Or something similar to that... > > > If we can't do that then perhaps the samplecontainer should have its > own > > > synd file somehow... or maybe in the sample container html init we > > should > > > just reset the parentRelayUrl to whatever it needs to be? > > > > > > (basically I just don't like changing the docs telling people it won't > > > work > > > :) > > > > > > - Cassie > > > > > > > > > On Fri, Mar 7, 2008 at 1:31 AM, Arne Roomann-Kurrik <[EMAIL PROTECTED] > > > > > wrote: > > > > > > > I figured that this might be the case. Which is preferable then? > If > > we > > > > leave it as-is, I could submit a patch for the sample container docs > > to > > > > document the need to change this value. > > > > > > > > ~Arne > > > > > > > > > > > > On Thu, Mar 6, 2008 at 4:23 PM, Zhen Wang <[EMAIL PROTECTED]> wrote: > > > > > > > > > The default relay URL was changed because it broke some > > non-opensocial > > > > > sample containers which pass parent parameters with a trailing > > slash. > > > > > > > > > > On Thu, Mar 6, 2008 at 4:14 PM, Kevin Brown <[EMAIL PROTECTED]> > wrote: > > > > > > Looks like an oversight. It was previously just > > files/container/..., > > > > but > > > > > > that introduced issues with different parent parameters. > > > > > > > > > > > > I think the default value should be /gadgets/files/... so that > the > > > > "out > > > > > of > > > > > > the box" setup is fine, and it only has to be changed if you > > change > > > > your > > > > > > context path manually. > > > > > > > > > > > > Zhen? > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Mar 6, 2008 at 4:07 PM, Arne Roomann-Kurrik < > > > [EMAIL PROTECTED] > > > > > > > > > > > wrote: > > > > > > > I'm trying to put together a demo of shindig where I download > > the > > > > > source > > > > > > > from SVN, build and run the server, and display a gadget in > the > > > > > OpenSocial > > > > > > > sample container - hopefully under 10 minutes. Since I've > been > > > > > playing > > > > > > > around with the sample container, I've noticed that calls to > > > > > > rpc_relay.html > > > > > > > are being directed to > > > > > > > > > > > > > > > > > > > > > > > > > http:// > > > > > > > > > > <my_server>:8080/gadgets/gadgets/files/container/rpc_relay.html#.................. > > > > > > > > > > > > > > (note the double gadgets path) which results in a 404. I > poked > > > > around > > > > > a > > > > > > bit > > > > > > > and found that the "parentRelayUrl" configuration property in > > > > > > > /config/syndicator.js is relatively pathed - prefixing a "/" > > > > provides > > > > > the > > > > > > > correct response (an "Unknown RPC service" error, but that's > > > > > expected). > > > > > > > > > > > > > > Is there a reason for the relative path in the configuration > > file? > > > > I > > > > > > > understand that this is just for testing and will be changed > in > > a > > > > > > production > > > > > > > setting, but changing it in the source will make the "out of > > box" > > > > > > experience > > > > > > > a bit smoother, IMO. > > > > > > > > > > > > > > Thanks, > > > > > > > ~Arne > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > ~Kevin > > > > > > > > > > > > > > > > > > > > -- > > ~Kevin > > > -- ~Kevin
