Comment #3 on issue 807 by [email protected]: FilterDefinition should wrap the servlet request like ServletDefinition
http://code.google.com/p/google-guice/issues/detail?id=807

I tried that... but (using my above example) the result is:

Request URI: /context/dir/com/google/whatever.java

I have probably 10 filters each with different paths. Properly identifying pathinfo is a critical step. I could make assumptions and hack on the returned urls, but that is not a good solution.

guice-servlet has the correct servlet path info internally, but the wrapped request isn't constructed correctly for filters. The wrapped request is correct (AFAIK) for servlets [1].

I think I now have a working dev environment. I can attempt a patch (bringing some of the ServletDefinition design to the FilterDefinition), if that would be welcome.

[1]: https://code.google.com/p/google-guice/source/browse/extensions/servlet/src/com/google/inject/servlet/ServletDefinition.java#194

--
You received this message because this project is configured to send all issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
"google-guice-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/google-guice-dev.
For more options, visit https://groups.google.com/d/optout.

Reply via email to