Bug 104153 - DefinePage.java throws away information from a POST.
DefinePage.java throws away information from a POST.
Product: Red Hat Web Application Framework
Classification: Retired
Component: ui (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Archit Shah
Jon Orris
Depends On:
Blocks: 99105
  Show dependency treegraph
Reported: 2003-09-10 11:28 EDT by Randy Graebner
Modified: 2007-04-18 12:57 EDT (History)
0 users

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2004-01-23 19:20:36 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Randy Graebner 2003-09-10 11:28:51 EDT
Description of problem:
DefinePage.java throws away information from a POST.

The patch is below but without it using bebop jsp with a POST causes no
information to be submitted.

[randyg@graebner intragroupe]$ diff -u
--- intragroupe/src/com/arsdigita/bebop/jsp/DefinePage.java	Wed Sep 10 11:16:56 2003
+++ core-platform/src/com/arsdigita/bebop/jsp/DefinePage.java	Wed Sep 10
11:17:41 2003
@@ -175,12 +175,6 @@
 	      HttpServletResponse resp =
-          String rct = null;
-          if (req.getMethod().toUpperCase().equals("POST")
-              && ((rct = req.getContentType()) != null)
-              && (rct.toLowerCase().startsWith("multipart"))) {
-              req = DispatcherHelper.restoreRequestWrapper(req);
-          }
 	      Document doc;
 	      PageState state;

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
Actual results:

Expected results:

Additional info:
Comment 1 Daniel Berrange 2003-09-10 11:39:04 EDT
NB, now we've moved to Servlet 2.3 for Rickshaw we should make
MultipartHttpServletRequest subclass HttpServletRequestWrapper. This will let us
forward onto other servlets without having to first unwrap our requests. Thus a
whole heap of dodgy code in DispatcherHelper, BaseDispatcherServlet, etc will go

From API docs for RequestDispatcher#forward 

The request and response parameters must be either the same objects as were
passed to the calling servlet's service method or be subclasses of the
ServletRequestWrapper or ServletResponseWrapper classes that wrap them.
Comment 2 Archit Shah 2003-10-16 16:37:10 EDT
this patch has been integrated onto 6.0 branch (@37048) and trunk (@37047).
Dan's suggestion has not yet been implemented, so this bug is moving to the
Rickshaw tracker ticket from the troika errata tracker.
Comment 3 Archit Shah 2004-01-23 19:20:36 EST
appplied to 5.2.x before 5.2.2 release (@39082)

Note You need to log in before you can comment on or make changes to this bug.