Closed GoogleCodeExporter closed 9 years ago
Looks like a problem at this place, as it wraps original Request but calculates
the pathInfo (should be decoded) using requestUrl (is not decoded):
https://code.google.com/p/google-guice/source/browse/extensions/servlet/src/com/
google/inject/servlet/ServletDefinition.java?r=389cb718a70cd11fdf9c336209246ebcf
e944755#205
Original comment by t.cserve...@gmail.com
on 13 Feb 2014 at 8:53
Proposed patch
https://github.com/sonatype/sisu-guice/pull/8
Original comment by t.cserve...@gmail.com
on 14 Feb 2014 at 11:40
I have same issue with it.
Gitiles gerrit plugin is not working well because of this issue.
Please refer following discussion.
https://groups.google.com/forum/#!topic/repo-discuss/px6X8_HOMPQ
Original comment by ukjung....@gmail.com
on 20 Feb 2014 at 4:01
BTW in the HttpServletRequest instances provided by Guice, getPathInfo()
consists of an URI-encoded path *and the query string*. The spec requires that
getPathInfo() only returns a de-coded path.
@Guice project: Is it normal that there no reaction from the project committers
to such a major bug? And is it normal that people fork the project (e.g.
sisu-guice) just to get bugs fixed?
Original comment by tobias.o...@sap.com
on 9 Oct 2014 at 11:54
The code.google.com guice project has migrated to GitHub. This issue site is
no longer being used. Please use https://github.com/google/guice/issues/745
instead.
Original comment by sberlin
on 9 Oct 2014 at 1:21
Original issue reported on code.google.com by
m...@hillsdon.net
on 11 Apr 2013 at 10:52