Cool, so container name it is; I'll go add that to the php code too then.

Thanks for the feedback!

On Mon, Mar 9, 2009 at 7:41 PM, Louis Ryan <[email protected]> wrote:

> The container was added to the BasicSecurityToken implementation in
> Shindig.
> Access to the container name is needed to read config values such as which
> OAuth store to use, or what the set of supported fields are for REST/RPC
>
> On Mon, Mar 9, 2009 at 6:03 AM, Chris Chabot <[email protected]> wrote:
>
> > I got a patch to fix this by reverting the security token back to the 6
> > entries:
> > Index: javascript/samplecontainer/samplecontainer.js
> > ===================================================================
> > --- javascript/samplecontainer/samplecontainer.js    (revision 747363)
> > +++ javascript/samplecontainer/samplecontainer.js    (working copy)
> > @@ -81,7 +81,7 @@
> >     for (var i = 0; i < gadgetUrl.length; i++) {
> >       appId += gadgetUrl.charCodeAt(i);
> >     }
> > -    var fields = [ownerId, viewerId, appId, "shindig", gadgetUrl, "0",
> > "default"];
> > +    var fields = [ownerId, viewerId, appId, "shindig", gadgetUrl, "0"];
> >     for (var i = 0; i < fields.length; i++) {
> >       // escape each field individually, for metachars in URL
> >       fields[i] = escape(fields[i]);
> >
> >
> > However since I have no idea if that last "default" is used for anything
> in
> > the java version I'm hesitant to just nuke it ... So does anyone know why
> > it's there and if I should start expecting 7 values on the token, or if
> we
> > can remove the 7th entry?
> >
> > background: php-shindig detects if it's an encrypted token or not by
> > splitting the string on ':', and if that results in exactly 6 parts, it
> > assumes it's a plain text token, hence the breaking on 7 :)
> >
> >  -- Chris
> >
> > ---------- Forwarded message ----------
> > From: Jakub Vrána (JIRA) <[email protected]>
> > Date: Tue, Feb 24, 2009 at 2:56 PM
> > Subject: [jira] Updated: (SHINDIG-931) Sample Container example doesn't
> > work
> > To: [email protected]
> >
> >
> >
> >    [
> >
> >
> https://issues.apache.org/jira/browse/SHINDIG-931?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
> > ]
> >
> > Jakub Vrána updated SHINDIG-931:
> > --------------------------------
> >
> >    Attachment: shindig-931.diff
> >
> > The problem is that samplecontainer.js generates 7 fields to token while
> > the
> > classes expects only 6 fields. I don't know what the 7th field "default"
> > could mean.
> >
> > > Sample Container example doesn't work
> > > -------------------------------------
> > >
> > >                 Key: SHINDIG-931
> > >                 URL: https://issues.apache.org/jira/browse/SHINDIG-931
> > >             Project: Shindig
> > >          Issue Type: Bug
> > >          Components: PHP
> > >    Affects Versions: trunk
> > >            Reporter: Jakub Vrána
> > >             Fix For: trunk
> > >
> > >         Attachments: shindig-931.diff
> > >
> > >
> > > Example at
> > http://shindig/gadgets/files/samplecontainer/samplecontainer.html or
> >
> >
> http://modules.partuza.nl/gadgets/files/samplecontainer/samplecontainer.htmlends
> > by "data.get("viewerFriends").getData() is null" JavaScript error.
> >
> > --
> > This message is automatically generated by JIRA.
> > -
> > You can reply to this email to add a comment to the issue online.
> >
>

Reply via email to