You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@shindig.apache.org by Sean Lin <se...@gmail.com> on 2009/02/03 11:23:09 UTC
RESTful support in java shindig
Hi,
What was the reason the handling of jsonBatch was removed from java
shindig? What is the current story with REST support?
Cheers,
Sean
Re: RESTful support in java shindig
Posted by Chris Chabot <ch...@google.com>.
Ah yeah that's a known issue unfortunately, since we switched to the RPC
protocol we haven't really bothered to update the REST implementation's
javascript implementation.
Thats to say, it's probably simple to fix, but no one with the correct
background has found it urgent enough to actually go and do it :)
I think we should really just remove the 'impl' config key, and always
assume RPC since it's presence does lead to a lot of confusion.
-- Chris
On Tue, Feb 3, 2009 at 4:11 PM, Sean Lin <se...@gmail.com> wrote:
> That looks right. It builds ok if impl is "rpc" but once it is changed to
> "rest" it will fail.
>
> -----Original Message-----
> From: Chris Chabot [mailto:chabotc@google.com]
> Sent: Tuesday, 3 February, 2009 10:32 PM
> To: shindig-dev@incubator.apache.org
> Subject: Re: RESTful support in java shindig
>
> Maybe the container.js config file hasn't been updated since the local
> changes conflict with svn, and thus your using an invalid config?
>
> That bit of config should read something like:
> "opensocial-0.8" : {
> // Path to fetch opensocial data from
> // Must be on the same domain as the gadget rendering server
> "impl" : "rpc", //Use "rpc" to enable JSON-RPC, "rest' for REST
> "path" : "http://%host%/social",
> "domain" : "shindig",
> "enableCaja" : false,
> "supportedFields" : {
> "person" : ["id", {"name" : ["familyName", "givenName",
> "unstructured"]}, "thumbnailUrl", "profileUrl"],
> "activity" : ["id", "title"]
> }
> }
>
>
>
> On Tue, Feb 3, 2009 at 3:22 PM, Sean Lin <se...@gmail.com> wrote:
>
> > Hmmm. I am getting test failures (can't fetch person info) when trying to
> > build java shindig when setting impl = rest in container.js. It seems
> that
> > there is still a reference to jsonBatch in restfulcontainer.js. Or is
> this
> > some misconfiguration on my part?
> >
> > -----Original Message-----
> > From: Chris Chabot [mailto:chabotc@google.com]
> > Sent: Tuesday, 3 February, 2009 7:25 PM
> > To: shindig-dev@incubator.apache.org
> > Subject: Re: RESTful support in java shindig
> >
> > jsonBatch was an old, undocumented, non-spec standard that we used before
> > we
> > actually had a good spec in place; So once we defined the JSON-RPC spec
> in
> > 0.8.1, we switched over to that.
> >
> > I'm not sure what the 'current story' is that your asking about, other
> then
> > 'Yes it supports both the REST and RPC end points'.
> >
> > On Tue, Feb 3, 2009 at 11:23 AM, Sean Lin <se...@gmail.com> wrote:
> >
> > > Hi,
> > >
> > > What was the reason the handling of jsonBatch was removed from java
> > > shindig? What is the current story with REST support?
> > >
> > >
> > >
> > > Cheers,
> > >
> > > Sean
> > >
> > >
> >
> >
>
>
RE: RESTful support in java shindig
Posted by Sean Lin <se...@gmail.com>.
That looks right. It builds ok if impl is "rpc" but once it is changed to "rest" it will fail.
-----Original Message-----
From: Chris Chabot [mailto:chabotc@google.com]
Sent: Tuesday, 3 February, 2009 10:32 PM
To: shindig-dev@incubator.apache.org
Subject: Re: RESTful support in java shindig
Maybe the container.js config file hasn't been updated since the local
changes conflict with svn, and thus your using an invalid config?
That bit of config should read something like:
"opensocial-0.8" : {
// Path to fetch opensocial data from
// Must be on the same domain as the gadget rendering server
"impl" : "rpc", //Use "rpc" to enable JSON-RPC, "rest' for REST
"path" : "http://%host%/social",
"domain" : "shindig",
"enableCaja" : false,
"supportedFields" : {
"person" : ["id", {"name" : ["familyName", "givenName",
"unstructured"]}, "thumbnailUrl", "profileUrl"],
"activity" : ["id", "title"]
}
}
On Tue, Feb 3, 2009 at 3:22 PM, Sean Lin <se...@gmail.com> wrote:
> Hmmm. I am getting test failures (can't fetch person info) when trying to
> build java shindig when setting impl = rest in container.js. It seems that
> there is still a reference to jsonBatch in restfulcontainer.js. Or is this
> some misconfiguration on my part?
>
> -----Original Message-----
> From: Chris Chabot [mailto:chabotc@google.com]
> Sent: Tuesday, 3 February, 2009 7:25 PM
> To: shindig-dev@incubator.apache.org
> Subject: Re: RESTful support in java shindig
>
> jsonBatch was an old, undocumented, non-spec standard that we used before
> we
> actually had a good spec in place; So once we defined the JSON-RPC spec in
> 0.8.1, we switched over to that.
>
> I'm not sure what the 'current story' is that your asking about, other then
> 'Yes it supports both the REST and RPC end points'.
>
> On Tue, Feb 3, 2009 at 11:23 AM, Sean Lin <se...@gmail.com> wrote:
>
> > Hi,
> >
> > What was the reason the handling of jsonBatch was removed from java
> > shindig? What is the current story with REST support?
> >
> >
> >
> > Cheers,
> >
> > Sean
> >
> >
>
>
Re: RESTful support in java shindig
Posted by Chris Chabot <ch...@google.com>.
Maybe the container.js config file hasn't been updated since the local
changes conflict with svn, and thus your using an invalid config?
That bit of config should read something like:
"opensocial-0.8" : {
// Path to fetch opensocial data from
// Must be on the same domain as the gadget rendering server
"impl" : "rpc", //Use "rpc" to enable JSON-RPC, "rest' for REST
"path" : "http://%host%/social",
"domain" : "shindig",
"enableCaja" : false,
"supportedFields" : {
"person" : ["id", {"name" : ["familyName", "givenName",
"unstructured"]}, "thumbnailUrl", "profileUrl"],
"activity" : ["id", "title"]
}
}
On Tue, Feb 3, 2009 at 3:22 PM, Sean Lin <se...@gmail.com> wrote:
> Hmmm. I am getting test failures (can't fetch person info) when trying to
> build java shindig when setting impl = rest in container.js. It seems that
> there is still a reference to jsonBatch in restfulcontainer.js. Or is this
> some misconfiguration on my part?
>
> -----Original Message-----
> From: Chris Chabot [mailto:chabotc@google.com]
> Sent: Tuesday, 3 February, 2009 7:25 PM
> To: shindig-dev@incubator.apache.org
> Subject: Re: RESTful support in java shindig
>
> jsonBatch was an old, undocumented, non-spec standard that we used before
> we
> actually had a good spec in place; So once we defined the JSON-RPC spec in
> 0.8.1, we switched over to that.
>
> I'm not sure what the 'current story' is that your asking about, other then
> 'Yes it supports both the REST and RPC end points'.
>
> On Tue, Feb 3, 2009 at 11:23 AM, Sean Lin <se...@gmail.com> wrote:
>
> > Hi,
> >
> > What was the reason the handling of jsonBatch was removed from java
> > shindig? What is the current story with REST support?
> >
> >
> >
> > Cheers,
> >
> > Sean
> >
> >
>
>
RE: RESTful support in java shindig
Posted by Sean Lin <se...@gmail.com>.
Hmmm. I am getting test failures (can't fetch person info) when trying to build java shindig when setting impl = rest in container.js. It seems that there is still a reference to jsonBatch in restfulcontainer.js. Or is this some misconfiguration on my part?
-----Original Message-----
From: Chris Chabot [mailto:chabotc@google.com]
Sent: Tuesday, 3 February, 2009 7:25 PM
To: shindig-dev@incubator.apache.org
Subject: Re: RESTful support in java shindig
jsonBatch was an old, undocumented, non-spec standard that we used before we
actually had a good spec in place; So once we defined the JSON-RPC spec in
0.8.1, we switched over to that.
I'm not sure what the 'current story' is that your asking about, other then
'Yes it supports both the REST and RPC end points'.
On Tue, Feb 3, 2009 at 11:23 AM, Sean Lin <se...@gmail.com> wrote:
> Hi,
>
> What was the reason the handling of jsonBatch was removed from java
> shindig? What is the current story with REST support?
>
>
>
> Cheers,
>
> Sean
>
>
Re: RESTful support in java shindig
Posted by Chris Chabot <ch...@google.com>.
jsonBatch was an old, undocumented, non-spec standard that we used before we
actually had a good spec in place; So once we defined the JSON-RPC spec in
0.8.1, we switched over to that.
I'm not sure what the 'current story' is that your asking about, other then
'Yes it supports both the REST and RPC end points'.
On Tue, Feb 3, 2009 at 11:23 AM, Sean Lin <se...@gmail.com> wrote:
> Hi,
>
> What was the reason the handling of jsonBatch was removed from java
> shindig? What is the current story with REST support?
>
>
>
> Cheers,
>
> Sean
>
>