Re: Accessing Protected Resources (ie images)

2012-03-08 Thread John Hjelmstad
Tricky situation, but not wholly uncommon. A few random thoughts: 1. Add an async gadgets.io.getProxyUrl() call. Painful, but would allow for a roundtrip to Shindig to pick up one or more signed URLs (or URLs w/ token attached). 2. Do the URLs *need* to be dynamically generated? If not, you can ha

Re: Emeritus status for inactive members

2011-12-16 Thread John Hjelmstad
Can I call quorum on this one? :) On Thu, Dec 15, 2011 at 9:12 PM, Gagandeep singh wrote: > +1 > > > On Thu, Dec 15, 2011 at 11:58 PM, Mark Weitzel >wrote: > > > +1 > > > > On Wed, Dec 14, 2011 at 10:49 AM, Henry Saputra > >wrote: > > > > > +1 > > > > > > On Tuesday, December 13, 2011, Paul Lin

Re: Emeritus status for inactive members

2011-12-13 Thread John Hjelmstad
+1 Though by writing this +1, does that mean my 2 years are refreshed? :) On Tue, Dec 13, 2011 at 7:43 AM, Bastian Hofmann wrote: > +1 > > On Tue, Dec 13, 2011 at 4:15 PM, Ciancetta, Jesse E. > wrote: > > +1 > > > >>-Original Message- > >>From: Paul Lindner [mailto:lind...@inuus.com] >

Re: Including Firebug Lite As A Feature In Shindig-Extras

2011-09-29 Thread John Hjelmstad
2c, I've been hoping someone would get around to this idea at some point. +1! On Wed, Sep 28, 2011 at 1:20 PM, Ryan J Baxter wrote: > Firebug lite is a great tool when you are using browsers that do not have > firebug. Including it as a feature would allow containers and gadgets to > request th

Re: Lcoked domains and proxy requests in shindig

2011-09-23 Thread John Hjelmstad
xml#gadgets.io.makeRequest > This api is meant to be used for all xhr transactions. It has mechanisms > to specify authentication (OAUTH, mostly) that will automatically get > applied to the request going out. There are currently no API's to build a > request URL to this endpoint and

Re: Lcoked domains and proxy requests in shindig

2011-09-21 Thread John Hjelmstad
the one the gadget is rendered in (locked). Do you set > cors header to allow all gadgets to reach your proxy URI? If so, what about > older browsers? > > > > tively > -John Hjelmstad wrote: - > > === > To: Dan Dumont/Westford/IBM@Lotus &

Re: Lcoked domains and proxy requests in shindig

2011-09-21 Thread John Hjelmstad
On Wed, Sep 21, 2011 at 4:08 PM, Dan Dumont wrote: > Hi John, Craig > > You mentioned some interesting things about the proxy in your thread, and I > didn't want to hijack it, but I have some tangent questions of my own. > > In shindig I see: > ProxyServlet.java > ... > // *TODO*: Consider

Re: Content-Disposition: attachment;filename=p.txt

2011-09-21 Thread John Hjelmstad
This is most importantly used by the proxy service. Without this handler, it would be trivial to hack content on another domain by an attacking context opening an IFRAME to a URL like: http://locked-domain.mygadgetdomain.com/gadgets/proxy?url=http://www.evil.com/read-data.html The content would c

Re: Review Request: Securing Features and RPC

2011-09-06 Thread John Hjelmstad
On Tue, Sep 6, 2011 at 5:25 PM, Ryan Baxter wrote: > > > > On 2011-09-06 21:13:01, johnfargo wrote: > > > Hey Ryan, > > > > > > The original intent of the fooKey syntax > is that it is simply a declaration that the feature has in it somwhere: > > > gadgets.rpc.register("fooKey", /* impl */); > >

Re: Securing RPC

2011-09-06 Thread John Hjelmstad
each feature a gadget requires in order to generate that list, but that > doesn't seem like that is the intent of exports with the type="rpc". > Sounds like we may need another export type to export the rpc services a > feature calls, would you agree? > > -Ryan > > Em

Re: Gadget To Gadget RPC

2011-09-02 Thread John Hjelmstad
gt; would be the container page? > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks Profile > > > > From: John Hjelmstad > To: dev@shindig.apache.org, > Date: 09/01/2011 04:11 PM > Subject:Re: Gadget To Gadget RP

Re: Gadget To Gadget RPC

2011-09-01 Thread John Hjelmstad
hat > top-level iFrame, correct? Is this what the relay URL is in the RPC code? > Or is the relay URL just used for cross domain container to gadget > communication? > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks Profile > > > > From

Re: Gadget To Gadget RPC

2011-09-01 Thread John Hjelmstad
Hey Ryan, You've pretty much hit the fundamental difficulty with any non-direct in-page communication: the intermediary always has some degree of control. Like you're saying, what you need is either an ID service of some kind, or a reliable convention in the context in which you're operating, to

Re: Securing RPC

2011-08-23 Thread John Hjelmstad
eloperworks/mydeveloperworks/profiles/user/RyanJBaxter> > > > > From:John Hjelmstad > To:Ryan J Baxter/Westford/IBM@Lotus, > Cc:dev@shindig.apache.org, Andrew E Davis < > andrew_e_davis%ib...@notesdev.ibm.com>,

Re: Securing RPC

2011-08-11 Thread John Hjelmstad
On Wed, Aug 10, 2011 at 5:16 PM, Ryan J Baxter wrote: > Responses inline. > > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks > Profile<http://www.ibm.com/developerworks/mydeveloperworks/profiles/user/RyanJBaxter> > >

Re: Securing RPC

2011-08-10 Thread John Hjelmstad
Hey Ryan, A few comments inline. On Wed, Aug 10, 2011 at 3:02 PM, Ryan J Baxter wrote: > John and Paul, > > In May, when Andrew and I were out at Google, we talked to you guys at a > high level about how to secure features and the RPC functionality in > Shindig. Andrew and I are at the point

Re: Review Request: RPC calls coming from iFrames within the same domain when using the same site id fail

2011-08-10 Thread John Hjelmstad
Hey Ryan, I'm taking a look @ the review shortly. Re: using window.frames, the original reason was basically laziness. For parent-to-child communication, we could do getElementById('id').contentWindow where available. -j On Tue, Aug 9, 2011 at 11:02 AM, Ryan J Baxter wrote: > John, in addition

Re: Review Request: Enable closure compiler by default

2011-08-02 Thread John Hjelmstad
Re: testing, I haven't done so -- just commenting on the code alone. Re: "var", removing it probably fixes our issue. I'm checking it out ;) On Tue, Aug 2, 2011 at 1:38 PM, Dan Dumont wrote: > > > > On 2011-08-02 20:06:10, johnfargo wrote: > > > Looks pretty good to me, w/ the caveat that Shindi

Re: Review Request: Split deferJs() functionality into its own method

2011-08-02 Thread John Hjelmstad
Thanks! Will commit after yours :) On Tue, Aug 2, 2011 at 1:09 PM, Michael Hermanto wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1237/#review1271 > --

Re: Review Request: Split deferJs() functionality into its own method

2011-08-02 Thread John Hjelmstad
Good catch, done On Tue, Aug 2, 2011 at 1:07 PM, Michael Hermanto wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/1237/#review1270 >

Re: None of the examples are working in HEAD

2011-08-01 Thread John Hjelmstad
I don't think it has to do with those changes, but may have to do with some others that I made last week to exportJs(). I'm looking into this now. On Sun, Jul 31, 2011 at 7:10 PM, Ryan J Baxter wrote: > Phil, I am also seeing this. > > John, could this have anything to do with these changes? > h

Re: Review Request: Abstracted 2 methods out.

2011-07-29 Thread John Hjelmstad
Agreed, well worth asking. On Fri, Jul 29, 2011 at 4:50 PM, Henry Saputra wrote: > Thanks for the reply John. Just want to make sure we add customization > point to Shindig as necessary =) > > - Henry > > On Fri, Jul 29, 2011 at 2:36 PM, John Hjelmstad wrote: > > For va

Re: Review Request: Abstracted 2 methods out.

2011-07-29 Thread John Hjelmstad
For various reasons, we happen to use different global symbol names in our use of the config lib. That in turn has to do with the fact that internal to Google we've built a config library that simplifies the current model considerably: rather than injecting config at the end of JS, we inject at the

Re: javascript readability..

2011-07-28 Thread John Hjelmstad
exported things exported once when they > >> share > >> a scope full of private helpers and other state. It also helps convey > when > >> the setup of a particular part of the object graph is done. > >> Eg. > >> > >> shindig.foo = (function()

Re: javascript readability..

2011-07-28 Thread John Hjelmstad
exported things exported once when they > >> share > >> a scope full of private helpers and other state. It also helps convey > when > >> the setup of a particular part of the object graph is done. > >> Eg. > >> > >> shindig.foo = (function()

Re: javascript readability..

2011-07-26 Thread John Hjelmstad
or air soon, maybe we can talk about AMD format and > what that brings to the table. :) > > > > From: John Hjelmstad > To: dev@shindig.apache.org, > Date: 07/26/2011 04:43 PM > Subject:Re: javascript readability.. > > > > I still prefer statu

Re: Enhancing gadgets.log

2011-07-26 Thread John Hjelmstad
Seems like a good idea to me, assuming the amount of code is kept to a bare minimum. Logging code is included (or is intended to be included) all over the place. On Tue, Jul 26, 2011 at 1:37 PM, Ryan J Baxter wrote: > I would probably be calling the RPC handler in logAtLevel with the level > and

Re: javascript readability..

2011-07-26 Thread John Hjelmstad
I still prefer status quo, as it reads more like a proper class to me, while being less verbose and centralizing the exported method definitions in a single place. As well, this question's corollary is whether to convert all instantiable objects to the form: FooClass.prototype.method = function()

Re: [VOTE] Ryan Baxter as Apache Shindig Committer

2011-07-22 Thread John Hjelmstad
+1 On Fri, Jul 22, 2011 at 10:20 AM, Michael Hermanto wrote: > +1 > > 2011/7/22 ๏̯͡๏ Jasvir Nagra > > > +1 (binding) > > > > On Fri, Jul 22, 2011 at 10:06 AM, Henry Saputra > >wrote: > > > > > +1 (binding) > > > > > > - Henry > > > > > > On Fri, Jul 22, 2011 at 8:51 AM, Paul Lindner > wrote: >

Re: Handling Certificates

2011-07-06 Thread John Hjelmstad
Where is the fact that you've OK'd a certificate stored in your system, and with what criteria? That is, is the cert supposed to be trusted on a per-MAC basis? In general, what you'd need in this situation sounds like it could be quite complex -- basically an HttpFetcher implementation that mirror

Re: Security token on iframe URL

2011-07-06 Thread John Hjelmstad
Hi Jacha: Security token is always templated right now: http://codesearch.google.com/#gYyO98kxBCI/carbon/dependencies/shindig/java/gadgets/src/main/java/org/apache/shindig/gadgets/uri/DefaultIframeUriManager.java&type=cs&l=155 Two solutions to this are: 1. Implement an injectable method for deter

Re: Felix Lee as Shindig Committer

2011-06-23 Thread John Hjelmstad
Woot, congrats! 2011/6/23 ๏̯͡๏ Jasvir Nagra > The votes are in. > > Congratulations Felix Lee on becoming a Shindig committer! > > On Wed, Jun 15, 2011 at 2:41 PM, Ziv Horesh wrote: > > > +1 > > > > On Wed, Jun 15, 2011 at 2:30 PM, Henry Saputra > >wrote: > > > > > +1 > > > > > > - Henry > > >

Re: RPC Transports Testing

2011-06-23 Thread John Hjelmstad
ll of the > transports. > > Thanks, > -Stanton > > > > From: John Hjelmstad > To: dev@shindig.apache.org, > Cc: johnfa...@gmail.com > Date: 06/22/2011 15:08 > Subject:Re: RPC Transports Testing > > > > Hi Stanton: > > The d

Re: RPC Transports Testing

2011-06-22 Thread John Hjelmstad
Hi Stanton: The description of browser support is in each of the *.transport.js files: http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/rpc/ It's relatively complete, but since the lib has evolved over time it can be summarized pretty easily: 1. window.postMessa

Re: Felix Lee as Shindig Committer

2011-06-15 Thread John Hjelmstad
+1 On Wed, Jun 15, 2011 at 11:09 AM, Mat Mannion wrote: > +1 > > 2011/6/15 ๏̯͡๏ Jasvir Nagra > > > Hello, > > > > I'd like to propose Felix Lee be granted committer access to Shindig > along > > with PMC membership, should he accept it. > > > > Felix has contributed substantially to Shindig's Ja

Re: Review Request: Create default AuthorityProvider to support host/port

2011-06-08 Thread John Hjelmstad
@Henry or @Paul, can you patch and commit this? LGTM but I'm swamped @ the moment :\ On Wed, Jun 8, 2011 at 10:06 AM, Li Xu wrote: > Hello, John/Stanton/Henry > That patch has been posted for a while...would you please help moving it > forward? thanks! > li > > [image: Inactive hide details for

Re: svn commit: r1102446 - in /shindig/trunk: config/ content/samplecontainer/examples/commoncontainer/ java/common/src/main/java/org/apache/shindig/common/servlet/ java/gadgets/src/main/java/org/apac

2011-05-20 Thread John Hjelmstad
li > [image: Inactive hide details for John Hjelmstad ---05/17/2011 06:25:39 > PM---Hi Li- We haven't open sourced the context-forking or inde]John > Hjelmstad ---05/17/2011 06:25:39 PM---Hi Li- We haven't open sourced the > context-forking or indeed our full > > > Fro

Re: svn commit: r1102446 - in /shindig/trunk: config/ content/samplecontainer/examples/commoncontainer/ java/common/src/main/java/org/apache/shindig/common/servlet/ java/gadgets/src/main/java/org/apac

2011-05-17 Thread John Hjelmstad
ile, I'd be happy to review a default Provider authorityProvider if you want to write it (using the code you had committed) and bound in DefaultGuiceModule or similar. --j On Tue, May 17, 2011 at 2:02 PM, Li Xu wrote: > [image: Inactive hide details for John Hjelmstad ---05/16/2011 06:47:05

Re: Review Request: Update to latest closure compiler.

2011-05-16 Thread John Hjelmstad
LGTM On Mon, May 16, 2011 at 6:06 PM, Paul Lindner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/746/ > --- > > (Updated 2

Re: svn commit: r1102446 - in /shindig/trunk: config/ content/samplecontainer/examples/commoncontainer/ java/common/src/main/java/org/apache/shindig/common/servlet/ java/gadgets/src/main/java/org/apac

2011-05-16 Thread John Hjelmstad
oving logic duplication by moving to ContainerConfig more cleanly. --j > > thanks, > li > > > > > > > [image: Inactive hide details for John Hjelmstad ---05/14/2011 01:33:11 > PM---Hi Li, Comments inline.]John Hjelmstad ---05/14/2011 01:33:11 PM---Hi > Li, Comments inline. > >

Re: svn commit: r1102446 - in /shindig/trunk: config/ content/samplecontainer/examples/commoncontainer/ java/common/src/main/java/org/apache/shindig/common/servlet/ java/gadgets/src/main/java/org/apac

2011-05-14 Thread John Hjelmstad
jected list of ConfigModifiers that can perform these edits. WDYT? --j > > thanks, > li > > > > > [image: Inactive hide details for John Hjelmstad ---05/13/2011 04:15:37 > PM---Hi all, This CL unfortunately suffers from a lot of static]John > Hjelmstad ---05/13/201

Re: svn commit: r1102446 - in /shindig/trunk: config/ content/samplecontainer/examples/commoncontainer/ java/common/src/main/java/org/apache/shindig/common/servlet/ java/gadgets/src/main/java/org/apac

2011-05-13 Thread John Hjelmstad
http://svn.apache.org/viewvc?rev=1102885&view=rev <http://svn.apache.org/viewvc?rev=1102885&view=rev>I'd be happy to work w/ you on moving the logic to ContainerConfig. --j On Fri, May 13, 2011 at 1:12 PM, John Hjelmstad wrote: > Hi all, > > This CL unfortunately

Re: svn commit: r1102446 - in /shindig/trunk: config/ content/samplecontainer/examples/commoncontainer/ java/common/src/main/java/org/apache/shindig/common/servlet/ java/gadgets/src/main/java/org/apac

2011-05-13 Thread John Hjelmstad
Hi all, This CL unfortunately suffers from a lot of static cling. There's no way to customize this behavior, and the default behavior ends up having odd side effects like getting and setting system properties, many of which are irrelevant, and tightly monitored, in production environments. I've t

Re: Add the ability to translate arbritary XML into JSON (issue4438071)

2011-05-05 Thread John Hjelmstad
Looks like it's a JSON-ordering issue. These are always painful. Error is: Results : Failed tests: jsonParse(org.apache.shindig.server.endtoend.EndToEndTest): test method jsonConvertXmlToJsonTest did not finish Running in Eclipse gives me: === EXCEPTION START Exception class=[net

Re: Issue : Microsoft Security Bulletin MS11-022

2011-05-05 Thread John Hjelmstad
Yeah, Nix is nixed. You need to switch to either RMR or Flash. --j On Thu, May 5, 2011 at 8:07 AM, Ciancetta, Jesse E. wrote: > Paul Lindner sent a message about this a few weeks back -- sounds like nix > is gone for good: > > http://markmail.org/message/cws5x6wbtpwrgt2a > > >-Original Mess

Re: troubleshooting flash.transport.js -- back port to 2.0.0 in progress

2011-05-03 Thread John Hjelmstad
On Tue, May 3, 2011 at 2:48 PM, Ed Venaglia wrote: > Hello all. > > I am encountering problems in implementing the flash transport in our > container. Our container is based the shindig 2.0.0 tag with a few, > very-minor, changes. > > Can someone help me resolve the problems I am encountering, so

Re: Enable the html caja serializer + tests (issue4441072)

2011-04-27 Thread John Hjelmstad
Happy to have "helped" On Wed, Apr 27, 2011 at 7:37 PM, wrote: > > > http://codereview.appspot.com/4441072/diff/3001/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaHtmlSerializer.java > File > > java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaHtmlSerial

Re: Add the ability to translate arbritary XML into JSON (issue4438071)

2011-04-26 Thread John Hjelmstad
I'm happy leaving it to you -- but these days I've been creating pretty fine-grained features. It's really just a matter of convenience vs. optimization in JS bundles. On Tue, Apr 26, 2011 at 4:09 PM, wrote: > Thanks for the comments John. I am fine with moving this functionality > out of the c

Re: DefaultIframeUriManager for locked domains

2011-04-04 Thread John Hjelmstad
ambiguously be parsed as the Authority. > > If this is going to cause too much turbulence, I could do some further > checking as was done before. > > Thoughts? > > Best regards, > -Stanton > > *(See attached file: DefaultIframeUriManagerPatch.txt)* > > [image: Inact

Re: DefaultIframeUriManager for locked domains

2011-04-04 Thread John Hjelmstad
@Stanton, the way Jacobo's describing is how we use it at Google -- we just rely on the browser to do the resolution. Adding scheme to the config doesn't work for us since we have containers (multiple actually) that run in both HTTP and HTTPS mode. We'd need a substitution var to make a single con

Re: DefaultIframeUriManager for locked domains

2011-03-31 Thread John Hjelmstad
that code. This > test produces LOCKED.lockeddomain.com as a domain and Uri.parse does not > blow up on that. However I did notice that the gadget url in the test is > null, so I am not sure if this test is valid since you can't create a hash > > of that. Stanton, we probabl

Re: DefaultIframeUriManager for locked domains

2011-03-31 Thread John Hjelmstad
Not when we run tests in Shindig... unless somehow that code path is never hit. On Thu, Mar 31, 2011 at 5:42 PM, Ziv Horesh wrote: > On Thu, Mar 31, 2011 at 4:18 PM, John Hjelmstad wrote: > > > Hi Stanton, > > > > Odd, this has worked for us for quite some time. Uri.par

Re: DefaultIframeUriManager for locked domains

2011-03-31 Thread John Hjelmstad
Hi Stanton, Odd, this has worked for us for quite some time. Uri.parse(...) just doesn't throw a UriException. Are you overriding UriParser? On Thu, Mar 31, 2011 at 2:57 PM, Stanton Sievers wrote: > Hi, > > Has anyone been able to use the DefaultIframeUriManager when locked > domains are configu

Java 1.5 vs 1.6 (java15 and java16 directories)

2011-03-29 Thread John Hjelmstad
Hello, With the recent commit of Paul Lindner's Maven pom.xml changes: http://svn.apache.org/viewvc?view=revision&revision=1085239 ...and my followup using these to allow us to utilize Closure for Java 1.6 JS compilation: http://svn.apache.org/viewvc?view=revision&revision=1085647 ...Shindig now

Re: What happened to the JsHandler?

2011-03-28 Thread John Hjelmstad
Yes, it has. It's been split into three JsProcessor classes. In order: 1. GetJsContentProcessor, which previously delegated to JsHandler. Now it does what it says: gets all the "raw" feature JS to process. 2. ConfigInjectionProcessor, which collects and injects gadgets.config.init({ ... }); inform

Re: HTTP / HTTPS question

2011-03-25 Thread John Hjelmstad
transition is probably not the right word. It doesn't always detect from > where it came http or https and always use the correct pathing. Does that > make sense? I guess I wilk try doing a replace all from http to https and > see what happens. > > On Mar 25, 2011 5:37 PM,

Re: Problem with circular dependencies in maven

2011-03-25 Thread John Hjelmstad
g Ellison >wrote: > > > I got the same error running the newest version when trying to do mvn > -Prun > > as well. > > > > On Mar 25, 2011 5:18 PM, "John Hjelmstad" wrote: > > > > Odd.. I just synced and didn't get a build error. > > > > 2011/3/25 ๏̯͡๏ Jasvir Nagra > I am seeing the following > > error when doing mvn -Pru... > > >

Re: HTTP / HTTPS question

2011-03-25 Thread John Hjelmstad
ccess all the > servlets through https... Or atleast I thought. I think my problem stems > from shindig only partially supporting https and http transitions. Is my > understanding wrong? If I did a find replace all on http to https might > that fix it? > > On Mar 25, 2011 5:02

Re: Problem with circular dependencies in maven

2011-03-25 Thread John Hjelmstad
Odd.. I just synced and didn't get a build error. 2011/3/25 ๏̯͡๏ Jasvir Nagra > I am seeing the following error when doing mvn -Prun: > > [INFO] > > [ERROR] BUILD FAILURE > [INFO] > -

Re: HTTP / HTTPS question

2011-03-25 Thread John Hjelmstad
Good reminder. We should move that to a Shindig-based solution IMO. Anyone able to volunteer? -j On Fri, Mar 25, 2011 at 3:59 PM, Niels van Dijk wrote: > > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > Hi, > > Another headsup when using https gadgets shindig currenlty uses > gmodules for r

Re: HTTP / HTTPS question

2011-03-25 Thread John Hjelmstad
default%3Ahttp%253A%2F%2Flocalhost%2Fxml%2FHelloWorld.xml%3A1%3Adefault > does not include scheme > > The gadget is a simple HelloWorld pulling viewer info > > > > > > > > > > > > > Further suggestions? > > > On Fri

Re: HTTP / HTTPS question

2011-03-25 Thread John Hjelmstad
0.48.73.232 is not allowed by > Access-Control-Allow-Origin. > > I'm not quite sure what its meaning. I think its saying its not > allowed because the call originated from https but then makes a call > to http regularly. Is that possible? > > > > On Fri, Mar 25, 2011 at

Re: HTTP / HTTPS question

2011-03-25 Thread John Hjelmstad
Hi Doug, Sounds like you might have an opensocial endpoint hard-coded to HTTP -- random guess. What browser are you using? I'd give Chrome a try and see what the Network tab says, to see if a request is being formulated. --j On Fri, Mar 25, 2011 at 11:12 AM, Doug Ellison wrote: > I'm running Ap

Re: Java 5/6 compatible builds

2011-03-24 Thread John Hjelmstad
That's my general understanding from Paul. It allows us (though I don't quite understand all the Maven juju) to write code and include dependencies against 1.6, while falling back to 1.5 if needed. One motivating use case for this in the immediate term is my and Michael's reintroduction of Closure

Re: preparing for shindig 3.0.0

2011-03-24 Thread John Hjelmstad
On Thu, Mar 24, 2011 at 9:01 AM, Paul Lindner wrote: > I'd like to get a beta build of shindig 3.0.0 ready soon. > > We've gone quite some time on trunk without a release and this would give > people a target they can start tracking. > > It will also flesh out the release process so I can documen

Re: Disable caching in gadgets.io.makeRequest() -- possible bug?

2011-03-14 Thread John Hjelmstad
Sounds technically like a grey area to me, but at the same time your approach seems reasonable. "nocache" in the rendering URL doesn't strictly mean that "nocache" should be appended to all downstream requests -- at least, AFAIK that's not spec'd anywhere. This said, what you've implemented is mo

Re: [jira] Updated: (SHINDIG-1516) Passing "rawxml" param w/o "url" param throws exception when processing spec

2011-03-11 Thread John Hjelmstad
Hi Dennis: quick update, could you post this to codereview.appspot.com? On Fri, Mar 11, 2011 at 1:19 PM, Dennis Ju (JIRA) wrote: > > [ > https://issues.apache.org/jira/browse/SHINDIG-1516?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel] > > Dennis Ju updated SHINDIG-1516:

Re: [ACTION REQUIRED] www.apache.org/dist/ housekeeping

2011-03-10 Thread John Hjelmstad
IMO we're in fairly good shape. But I don't have great insight into how many people are using which release branches. On Thu, Mar 10, 2011 at 12:37 AM, Gagandeep singh wrote: > Please correct me if im wrong. > Shindig seems to have 1.0.1 and 2.0.0 releases, so i guess we are ok. > Or should we be

Re: Render gadget by passing content instead of URL?

2011-03-09 Thread John Hjelmstad
; but perhaps an if clause should be added to bypass url validation if the > rawxml parameter is present? > > Thoughts? > > Thanks, > Dennis > > > On Tue, Mar 8, 2011 at 11:26 AM, John Hjelmstad wrote: > >> This is actually possible - you just POST to th

Re: MakeRequestHandler.java Question

2011-03-09 Thread John Hjelmstad
headerData in this case is a query param, ie &headers=foo%3Dbar%26containsequals%3Dsheen%25Dcrazy getParameter performs the first unescape to value: foo=bar&containsequals=sheen%3Dcrazy ...which is in turn split on "&" 1. foo=bar 2. containsequals=sheen%3Dcrazy each of these is processed into a

Re: Allow extensions to the common container by allowing other features to provide their own namespace (issue4260049)

2011-03-08 Thread John Hjelmstad
s.addContainerMixin = function(namespace, func){ >shindig.container.Container.prototype.mixins_[namespace] = func; > } > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks Profile > > > > From: John Hjelmstad > To: dev@

Re: Allow extensions to the common container by allowing other features to provide their own namespace (issue4260049)

2011-03-08 Thread John Hjelmstad
ner = new shindig.container.Container(); > container.foo.fooMethod(); > > The mixins method gets called by the constructor of Container and just > adds the methods to the Container object, but only if foo feature is > included on the page. > > Hope that makes sense. > > -Ryan

Re: Render gadget by passing content instead of URL?

2011-03-08 Thread John Hjelmstad
This is actually possible - you just POST to the rendering URL with param &rawxml=. Code location isn't intuitive. It works if you're using a GadgetSpecFactory that extends (or is) DefaultGadgetSpecFactory: http://codesearch.google.com/codesearch/p?hl=en#oS3Kom0H2nk/trunk/shindig/shindig_735392/j

Re: Allow extensions to the common container by allowing other features to provide their own namespace (issue4260049)

2011-03-08 Thread John Hjelmstad
t when it has not been created yet. I > want a feature to be able to add it's own methods to the Container object > in its own namespace. So the mixins_ variable would only be used by other > features. > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-304

Re: Allow extensions to the common container by allowing other features to provide their own namespace (issue4260049)

2011-03-08 Thread John Hjelmstad
Hi Ryan: The pattern of exposing a magical variable that's read when you call this.mixin_() seems a little odd to me. I'd prefer to have something more direct, ie have the mixin method itself take a method/function (which we could define as being bound to "this" if you like). --j On Tue, Mar 8,

Re: Invalid URL Pattern In web.xml

2011-03-04 Thread John Hjelmstad
Weird, I just added that and it worked fine when run with Jetty. Perhaps it's not technically Servlet compliant however. I'll change to something else. Thanks for the heads up- John On Fri, Mar 4, 2011 at 8:15 AM, Ryan J Baxter wrote: > Anyone else seeing this? Looks like its being caused by >

Re: Unit Tests For Features

2011-03-02 Thread John Hjelmstad
t; > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks Profile > > > > From: John Hjelmstad > To: dev@shindig.apache.org, > Cc: Ryan J Baxter/Westford/IBM@Lotus > Date: 03/02/2011 03:51 PM > Subject:Re: Unit Tests For Featur

Re: Unit Tests For Features

2011-03-02 Thread John Hjelmstad
> > -Ryan > > Email: rjbax...@us.ibm.com > Phone: <978-899-3041>978-899-3041 > developerWorks Profile > > > > From: John Hjelmstad > To: dev@shindig.apache.org, > Cc: Ryan J Baxter/Westford/IBM@Lotus > Date: 03/02/2011 01:59 PM > Sub

Re: svn commit: r1074780 - in /shindig/trunk: extras/src/main/javascript/features-extras/pubsub-2/ extras/src/main/javascript/features-extras/wave/ features/src/main/javascript/features/caja/ features

2011-02-28 Thread John Hjelmstad
ore", "glob" > and "shindig.auth". I didn't see this before because I just dumped the > resolved dependencies. > I will look how this is handled in the Java version and refactor it > accordingly. > > Thanks > > -- Bastian > > 2011/2/28 John

Re: svn commit: r1074780 - in /shindig/trunk: extras/src/main/javascript/features-extras/pubsub-2/ extras/src/main/javascript/features-extras/wave/ features/src/main/javascript/features/caja/ features

2011-02-28 Thread John Hjelmstad
Hi Bastian: Odd, feature=taming doesn't appear to depend on anything for me: http://svn.apache.org/repos/asf/shindig/trunk/features/src/main/javascript/features/taming/feature.xml Can you point out the loop you're seeing? Thx, John On Mon, Feb 28, 2011 at 6:18 AM, Bastian Hofmann wrote: > Hi,

Re: Hudson build became unstable: Shindig » Apache Shindig Web App #2217

2011-02-24 Thread John Hjelmstad
Fixed now... On Thu, Feb 24, 2011 at 11:56 AM, Apache Hudson Server < hud...@hudson.apache.org> wrote: > See < > https://hudson.apache.org/hudson/job/Shindig/org.apache.shindig$shindig-server/2217/ > > > > >

Re: Hudson build became unstable: Shindig #2217

2011-02-24 Thread John Hjelmstad
Should already be fixed... this appears to have been caused by the missing feature.txt update just committed. On Thu, Feb 24, 2011 at 11:56 AM, Apache Hudson Server < hud...@hudson.apache.org> wrote: > See > > >

Re: Hudson build became unstable: Shindig #2215

2011-02-24 Thread John Hjelmstad
Thanks Michael! On Wed, Feb 23, 2011 at 11:05 PM, Michael Hermanto wrote: > Fixed with http://svn.apache.org/viewvc?rev=1074048&view=rev > > On Wed, Feb 23, 2011 at 9:49 PM, Michael Hermanto >wrote: > > > On it. > > > > > > On Wed, Feb 23, 2011 at 9:14 PM, Apache Hudson Server < > > hud...@hudso

Re: Small change to render doctype (issue4172053)

2011-02-17 Thread John Hjelmstad
@Jasvir, perhaps let's make this behavior configurable, on an opt-in basis? On Thu, Feb 17, 2011 at 1:11 PM, wrote: > > > http://codereview.appspot.com/4172053/diff/1/java/gadgets/src/main/java/org/apache/shindig/gadgets/parse/caja/CajaHtmlSerializer.java > File > > java/gadgets/src/main/java/or

Re: Gadgets Spec and Databases

2011-02-08 Thread John Hjelmstad
The vast majority of people serve them on public servers. For enterprise use cases, I've heard of several solutions, but they typically boil down to storage as a blob in a distributed database, with a corresponding HttpFetcher that grabs 'em. --j On Tue, Feb 8, 2011 at 4:52 PM, Maxwell wrote:

Re: [VOTE] Approve Mat Mannion for Shindig Commiter/PMC

2011-02-07 Thread John Hjelmstad
+1 On Mon, Feb 7, 2011 at 2:29 PM, Paul Lindner wrote: > +1 > > On Mon, Feb 7, 2011 at 8:47 AM, Paul Lindner wrote: > > > Hello, > > > > I would like to propose that Mat Mannion be granted committer access to > > Shindig along with PMC membership. > > > > Mat has contributed a number of patches

Re: [VOTE] Approve Jacobo Tarrio as Shindig Committer

2011-02-07 Thread John Hjelmstad
+1 Voting on own proposal. On Mon, Feb 7, 2011 at 1:58 PM, Michael Hermanto wrote: > +1 > > On Mon, Feb 7, 2011 at 1:57 PM, John Hjelmstad wrote: > > > Hello, > > > > I would like to propose that Jacobo Tarrio be granted committer access > > to Shindig along

[VOTE] Approve Jacobo Tarrio as Shindig Committer

2011-02-07 Thread John Hjelmstad
Hello, I would like to propose that Jacobo Tarrio be granted committer access to Shindig along with PMC membership. Jacobo has contributed substantially to the Java Shindig effort, most recently in JavaScript serving but also in the Proxy API, and has done several code reviews and provided many b

Re: [VOTE] Approve Mat Mannion for Shindig Commiter/PMC

2011-02-07 Thread John Hjelmstad
+1 On Mon, Feb 7, 2011 at 10:24 AM, chico charlesworth < chico.charleswo...@gmail.com> wrote: > +1 > > On 7 February 2011 17:51, Gagandeep Singh wrote: > > > +1 > > > > Thanks > > Gagan > > > > -- > > The only thing missing in life is background music. > > -- Gagandeep Singh > > > > > > On Mon,

Re: Java build environment

2011-02-01 Thread John Hjelmstad
http://svn.apache.org/repos/asf/shindig/trunk/BUILD-JAVA still looks reasonably up-to-date/relevant. On Tue, Feb 1, 2011 at 5:40 PM, Xue-Feng Yang wrote: > I know there is an article "Building and running Apache Shindig for Java". > I > followed it step by step many times, but still failed at ce

Re: Build failed in Hudson: Shindig » Apache Shindig Features #2168

2011-02-01 Thread John Hjelmstad
FYI This should already be fixed. On Tue, Feb 1, 2011 at 5:19 PM, Apache Hudson Server < hud...@hudson.apache.org> wrote: > See < > https://hudson.apache.org/hudson/job/Shindig/org.apache.shindig$shindig-features/2168/changes > > > > Changes: > > [johnh] Creates feature="core.config.base", on wh

Re: Avoiding proxy and concat urls in gadget render.

2011-01-31 Thread John Hjelmstad
Not functionally. Content optimizing rewriters are intended to help a gadget developer scale their app w/o scaling their servers to handle the load. On Mon, Jan 31, 2011 at 4:20 PM, Maxwell wrote: > Worked, thanks. > > Are there any problem in disable the re > > On Mon, Jan 31, 2011 at 1:37 PM,

Re: CacheEnforcementVisitor is added to script & css concat rewriters

2011-01-31 Thread John Hjelmstad
LGTM On Sun, Jan 30, 2011 at 9:34 PM, Pulkit Goyal wrote: > Reviewers: dev-remai...@shindig.apache.org, dev@shindig.apache.org, > gagan.g...@gmail.com, lind...@inuus.com, johnfa...@gmail.com > > Description: > > CacheEnforcementRewriter is added to relevant rewriters so that only > non-private >

Re: Pub/Sub for different containers.

2011-01-18 Thread John Hjelmstad
in rpc, just like pubsub > does? > > is that flash public, are you putting it on shindig or other place? > > Thanks > > On Mon, Jan 10, 2011 at 10:40 PM, John Hjelmstad wrote: > > > Very late response, back from vacation. Gadget-to-gadget is typically > only >

Re: Class Loader Problems Within Shindig

2011-01-11 Thread John Hjelmstad
ript files in features, not in > the container? > > -Ryan > > Email: rjbax...@us.ibm.com > Phone: 978-899-3041 > developerWorks Profile > > > > From: John Hjelmstad > To: dev@shindig.apache.org > Date: 01/11/2011 03:15 PM > Subject:Re: Cl

Re: Class Loader Problems Within Shindig

2011-01-11 Thread John Hjelmstad
The approach you're describing sounds fine to me. Another place that uses the ResourceLoader class is the JS framework, for features with entries like --j On Tue, Jan 11, 2011 at 11:59 AM, Ryan J Baxter wrote: > Today I ran into a few problems with the class loader Shindig is using in > some o

Re: Pub/Sub for different containers.

2011-01-10 Thread John Hjelmstad
;> My test will be a simple chat (2 gadgets sending messages), it's easy > with pub/sub, i already wrote one, but now we would like to try with > different containers. > >>> > >>> > >>> Maxwell > >>> > >>> On Tue, Dec 21, 2010

Re: custom GadgetRewriter question

2011-01-10 Thread John Hjelmstad
er, templateRewriter, >>> absolutePathRewriter, styleTagExtractorRewriter, >>> styleAdjacencyRewriter, proxyingRewriter, >>> cajaRewriter, sanitizedRewriter, renderingRewriter, >>> i18nRewriter); >>> } >>> >>> What if I&

Re: ActivityStreams Social API

2011-01-06 Thread John Hjelmstad
No opposition here. Looking forward to the patches. On Wed, Jan 5, 2011 at 1:54 PM, Eric Woods wrote: > Hi Fellow Shindig Developers, > > OpenSocial 2.0 is upon us (targeting release in May). In preparation for > Shindig 3.0's support of OpenSocial 2.0, I'd like to start factoring the > Activit

Re: custom GadgetRewriter question

2010-12-21 Thread John Hjelmstad
IMO depends on how much customization you want to do. If you want to add a rewriter to the beginning or end of the rewriter chain you might just be able to extend RewriteModule. We @Google have enough customization that we basically just created our own RewriteModule and in it supply our own bindin

  1   2   3   4   >