users@grizzly.java.net

Re: Session management bug / Static resource as stream

From: Loic Petit <loic.petit_at_gmail.com>
Date: Tue, 2 Sep 2008 23:58:43 +0100

Yop,

> I was under the impression the code was working :-) :-) So with your
> patch, you make sure the contextPath and the ServletPath are
> compared against the original URI. But fullPathURI is computed as:
>
>> if (fullUrlPath == null){
>> fullUrlPath = contextPath + servletPath;
>> }
>
> What is the difference? Sorry that might be a stupid question....

The only difference is : I treat this thing as you do, the only
specific thing is : I do not throw any error message and I return 404
for the chain.
(btw the patch is only replacing customizeErrorPage(response,
"Resource Not Found", 404); by response.setStatus(404))

> BTW You don't have to call:
>
> request.getRequest().action(ActionCode.ACTION_POST_REQUEST, null);
>
> as StaticResourceAdapter.postService will do it for you:
>
> https://grizzly.dev.java.net/nonav/xref/com/sun/grizzly/tcp/StaticResourcesAdapter.html
> #210
>
> Same for res.finish();

Thanks for the advice

>
> This is a good recommendation. Sorry for asking again, but can you
> file an RFE? I suspect I can re-use the Jetty one (Apache2 licence).
> I just need to learn how Greg did it.
>
> Keep the recommendations coming!

No problem, I do it right away...

Enjoy !

Loïc