Re: Review Request: Need to support query parameter for Social REST API.

2012-02-06 Thread Ryan Baxter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3764/#review4854 --- /trunk/java/social-api/src/main/java/org/apache/shindig/social/openso

Re: [QUESTION] Any reason why the actions container mixin does not call Container.rpcRegister instead of gadgets.rpc.register?

2012-02-06 Thread Ryan J Baxter
Henry I think this is probably an oversight in the code review. I don't see why we couldn't use the container's API From: Henry Saputra To: dev@shindig.apache.org, Cc: Dan Dumont/Westford/IBM@Lotus Date: 02/06/2012 09:23 PM Subject:[QUESTION] Any reason why the act

[QUESTION] Any reason why the actions container mixin does not call Container.rpcRegister instead of gadgets.rpc.register?

2012-02-06 Thread Henry Saputra
Hi Guys, Any reason why the implementation of the actions container mixin does not call Container.rpcRegister but instead directly call gadgets.rpc.register? The osapi.container.Container.prototype.rpcRegister has special code to inject "gs" key to include the caller gadget site. Here is the sni

Re: How to config the rpc to a remote Shindig server

2012-02-06 Thread Henry Saputra
Looks like it needs code fix. Maybe we could use the client side shindig.uri feature to construct the right URL - Henry On Mon, Feb 6, 2012 at 3:09 PM, Li Xu wrote: > Stanton, > Doesn't seem like it can workaround without a fix this time...Here's what I > think how we may fix this: > > cfg[osapi

Re: How to config the rpc to a remote Shindig server

2012-02-06 Thread Li Xu
Stanton, Doesn't seem like it can workaround without a fix this time...Here's what I think how we may fix this: cfg[osapi.container.ServiceConfig.PROXY_URI] = "http://localhost:8181/proxy "; cfg[osapi.container.ServiceConfig.API_HOST] = "http://localhost:8080/cre";; cfg[osapi.container.ServiceCon

Re: Review Request: Need to support query parameter for Social REST API.

2012-02-06 Thread li xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3764/ --- (Updated 2012-02-06 21:47:43.400978) Review request for shindig and Eric Woods.

Re: Review Request: Need to support query parameter for Social REST API.

2012-02-06 Thread Stanton Sievers
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3764/#review4831 --- There seems to be trailing whitespace and some tabs in this patch. Ca

Re: Review Request: Maven project enhancements to improve embedding and extending shindig-server

2012-02-06 Thread Jesse Ciancetta
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3695/#review4830 --- Ship it! LGTM I applied the patch, ran the SVN move commands and was

FW: [PROPOSAL] Maven project enhancements to improve embedding and extending shindig-server (for Apache Rave)

2012-02-06 Thread Ciancetta, Jesse E.
Ate replied to me last week but it looks like it only went to rave-dev@ -- forwarding to dev@shindig too... >-Original Message- >From: Ate Douma [mailto:a...@douma.nu] >Sent: Friday, February 03, 2012 3:57 PM >To: rave-...@incubator.apache.org >Subject: Re: [PROPOSAL] Maven project enhanc

Review Request: Need to support query parameter for Social REST API.

2012-02-06 Thread li xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3764/ --- Review request for shindig and Eric Woods. Summary --- We have a use case

Re: How to config the rpc to a remote Shindig server

2012-02-06 Thread Grace
Thanks Stanton, It can work basically. But I found another problem with an encode issue, as the endpoint is simply plus the host and path. For example: cfg[osapi.container.ServiceConfig.API_HOST] = " http://localhost:8081/proxy?url=http//localhost:8080/cre/"; cfg[osapi.container.ServiceConfig.API_P