search

javaee / grizzly

Writing scalable server applications in the Java™ programming language has always been difficult. Before the advent of the Java New I/O API (NIO), thread management issues made it impossible for a server to scale to thousands of users. The Grizzly NIO framework has been designed to help developers to take advantage of the Java™ NIO API.
https://javaee.github.io/grizzly/
Other
222 stars 60 forks source link

When servicing push request, getPathInfo() includes query string #1900

Closed glassfishrobot closed 7 years ago

glassfishrobot commented 7 years ago

If the resource being pushed is dynamic, as in the case of JSF and its ResourceHandler, then the query string is incorrectly included when HttpServletRequest.getPathInfo() is called, this causes many problems.

Will attach reproducer.

glassfishrobot commented 7 years ago

Reported by @edburns

glassfishrobot commented 7 years ago

@edburns said:

Steps to reproduce

War approach

  1. Deploy the war
  2. Visit <https://localhost:8181/web-servlet-4.0-push-dynamic/test> in Chrome, saying yes to the security prompt.
  3. Observe that the resource with a query string yields a 500.

Source code approach

  1. svn update your glassfish-appserv-tests trunk
  2. Apply the attached patch
  3. cd devtests/web and run ant edburns
  4. Observe the test fails.
glassfishrobot commented 7 years ago

@rlubke said: Changes applied: 670de7fc52f36e947f524b087a82cf2d2e6deac9

glassfishrobot commented 7 years ago

File: 20170403-1955Z-GRIZZLY-1900-test.patch Attached By: @edburns

glassfishrobot commented 7 years ago

File: web-servlet-4.0-push-dynamic-web.war Attached By: @edburns

glassfishrobot commented 7 years ago

This issue was imported from java.net JIRA GRIZZLY-1900

glassfishrobot commented 7 years ago

Marked as fixed on Monday, April 3rd 2017, 1:10:37 pm