Re: Tapestry Problem on Google App Engine in Development

2012-10-23 Thread Lance Java
I'm not 100% sure that tapestry has set the header since response.containsHeader() returns false. It's probably best to debug to be sure. My gut feeling is that gae is having troubles with setting a header after writing to the response body. Although tapestry is doing this legally, under normal c

Re: Tapestry Problem on Google App Engine in Development

2012-10-23 Thread Thiago H de Paula Figueiredo
On Tue, 23 Oct 2012 12:37:38 -0200, Dan Cyr wrote: In summary: I am using Tapestry 5.3.5. In development, GAE 1.7.2 no longer provided the gzip header (1.7.1 used to) (see previous logs and debugs) Would tapestry be dependent on the servlet implementation? Is this a GAE issue? It sure

Re: Tapestry Problem on Google App Engine in Development

2012-10-23 Thread Dan Cyr
In summary: I am using Tapestry 5.3.5. In development, GAE 1.7.2 no longer provided the gzip header (1.7.1 used to) (see previous logs and debugs) Tapestry sends gzip anyways. Black diamonds in browser. Please improve any of my assumptions... Can anyone chime in and explain what may be happen

Re: Tapestry Problem on Google App Engine in Development

2012-10-19 Thread Howard Lewis Ship
There's a recently fixed bug where, once content is compressed, the compressed content is cached for later requests regardless of whether the client requests compression or not. Fixed in 5.3.5. https://issues.apache.org/jira/browse/TAP5-1868 On Fri, Oct 19, 2012 at 2:42 PM, Lance Java wrote: >

Re: Tapestry Problem on Google App Engine in Development

2012-10-19 Thread Lance Java
So, it looks like something about GAE 1.7.2 is causing tapestry to send gzipped content but not set the content encoding header. I'm baffled and out of ideas... anyone else got any ideas? -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engin

Re: Tapestry Problem on Google App Engine in Development

2012-10-19 Thread Dan Cyr
Interesting: @Override public boolean service(HttpServletRequest request, HttpServletResponse response, HttpServletRequestHandler handler) throws IOException { boolean rc = handler.service(request, response); logger.error("Contains header:

Re: Tapestry Problem on Google App Engine in Development

2012-10-19 Thread Lance Java
Is MyFilter printing out before or after handler.service(request, response)? You'll need to print it out after. Taking a look at the tapestry source code, the GZipFilter wraps the response in an implementation that will only apply GZipping when a certain threshold is reached (determined by the "ta

Re: Tapestry Problem on Google App Engine in Development

2012-10-17 Thread Dan Cyr
Does this look like the proper way? >From AppModule, public void contributeHttpServletRequestHandler( OrderedConfiguration configuration) { configuration.add("MyFilter", new MyFilter(), "after:GZIP"); } On Wed, Oct 17, 2012 at 9:30 AM, Lance Java wrote: > Hang on a

Re: Tapestry Problem on Google App Engine in Development

2012-10-17 Thread Lance Java
Hang on a sec... are you saying the containsHeader("Content-Encoding") is false even when it works on GAE 1.7.1? I don't think that you've configured your filter correctly considering your previous results below: 1.7.1 Response Headersview source Content-Encodinggzip Content-Typetext/ht

Re: Tapestry Problem on Google App Engine in Development

2012-10-17 Thread Lance Java
So, this seems to be an issue in tapestry after all. Can any tapestry gurus explain why tapestry would return gzipped content without setting the Content-Encoding header? -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Developmen

Re: Tapestry Problem on Google App Engine in Development

2012-10-17 Thread Dan Cyr
Oops, pardon me, I see now, can't get the value, only true/false. AppModule: @Override public boolean service(HttpServletRequest request, HttpServletResponse response, HttpServletRequestHandler handler) throws IOException { logger.error("Contains header: " + response.c

Re: Tapestry Problem on Google App Engine in Development

2012-10-17 Thread Lance Java
> .containsHeader(..yes, pre 3.0 So print the boolean instead of the value. I just wanna see if tapestry has populated it. -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Development-tp5716763p5716988.html Sent from the Tapest

Re: Tapestry Problem on Google App Engine in Development

2012-10-16 Thread Dan Cyr
.containsHeader(..yes, pre 3.0 .getHeader( 3.0 GAE must be using pre-3.0. Servlet spec... http://docs.oracle.com/javaee/6/api/javax/servlet/http/HttpServletResponse.html#getHeader%28java.lang.String%29 Sigh... getHeader java.lang.String *getHeader*(java.lang.String name) Gets the va

Re: Tapestry Problem on Google App Engine in Development

2012-10-16 Thread Lance Java
It's in the 2.3 javadocs? http://docs.oracle.com/cd/E17802_01/products/products/servlet/2.3/javadoc/javax/servlet/http/HttpServletResponse.html#containsHeader(java.lang.String) -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Develo

Re: Tapestry Problem on Google App Engine in Development

2012-10-16 Thread Dan Cyr
That also is Servlet spec 3.0... On Fri, Oct 12, 2012 at 11:37 AM, Lance Java wrote: > response.containsHeader("Content-Encoding") ;) > > > > -- > View this message in context: > http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Development-tp5716763p5716859.html > Se

Re: Tapestry Problem on Google App Engine in Development

2012-10-12 Thread Lance Java
response.containsHeader("Content-Encoding") ;) -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Development-tp5716763p5716859.html Sent from the Tapestry - User mailing list archive at Nabble.com. --

Re: Tapestry Problem on Google App Engine in Development

2012-10-12 Thread Dan Cyr
That also is Servlet spec 3.0... On Fri, Oct 12, 2012 at 11:27 AM, Lance Java wrote: > response.getHeader("Content-Encoding") > > > > -- > View this message in context: > http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Development-tp5716763p5716857.html > Sent from

Re: Tapestry Problem on Google App Engine in Development

2012-10-12 Thread Lance Java
response.getHeader("Content-Encoding") -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Problem-on-Google-App-Engine-in-Development-tp5716763p5716857.html Sent from the Tapestry - User mailing list archive at Nabble.com. --

Re: Tapestry Problem on Google App Engine in Development

2012-10-12 Thread Dan Cyr
Hmm..Where is my mind, of course.. But it seems response.getHeaderNames() is in Servlet Spec 3.0.Seems like GAE gwt-2.4.0/gwt-user.jar is 1.5 and doesn't have that call available? Another way to check? Anyway, I'm convinced it's a GAE 1.7.1 -> 1.7.2 upgrade problemPerhaps GAE is not worth

Re: Tapestry Problem on Google App Engine in Development

2012-10-12 Thread Lance Java
> logger.error(name + " " + request.getHeader(name)); I meant you to log the response headers, not the request headers. This way we can check if it's tapestry's fault for not including the header or GAE's. -- View this message in context: http://tapestry.1045711.n5.nabble.com/Tapestry-Proble

Re: Tapestry Problem on Google App Engine in Development

2012-10-11 Thread Dan Cyr
When I turn off gzip compression with configuration.add(SymbolConstants.GZIP_COMPRESSION_ENABLED, "false"); my page generates a java.lang.IllegalStateException Sanity check - neither a stream response nor a redirect response was generated for this action request. Can we at least get tapestry-de

Re: Tapestry Problem on Google App Engine in Development

2012-10-11 Thread Lance Java
I'm assuming that this is a problem in GAE but if you want to check, you could contribute a HttpServletRequestHandler to print out the response headers as set by tapestry. Contribute a filter with ordering "after:GZIP". NB. You can disable compression by setting SymbolConstants.GZIP_COMPRESSION_EN

Re: Tapestry Problem on Google App Engine in Development

2012-10-11 Thread Dan Cyr
There seems to be a "Content-Encodinggzip" missing on the response in 1.7.2. Would that be the cause? Should GAE provide that header? Are they listening or can we inform them? 1.7.1 Response Headersview source Content-Encodinggzip Content-Typetext/html; charset=utf-8 Server

Re: Tapestry Problem on Google App Engine in Development

2012-10-11 Thread Lance Java
Using firebug (or similar) can you look at the request/response traffic when the error occurs? In particular, what are the values of the "Accept-Encoding" header on the request and the "Content-Encoding" header on the response? -- View this message in context: http://tapestry.1045711.n5.nabble