Tapestry
  1. Tapestry
  2. TAPESTRY-340

url-encoded parameters are not considered in multipart-requests

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 4.0, 3.0.5
    • Fix Version/s: 4.1.3
    • Component/s: Framework
    • Labels:
      None
    • Environment:
      all

      Description

      Problem-setup:
      a component uses URL-encoded parameters inside a form leading to a multipart/mime-encoded request.

      Problem: all these parameters will be ignored (they're expected as parts of the multipart-request).

      Proposed fix:
      In the class org.apache.tapestry.request.RequestContext, replace the method getParameter()
      with the following;
      _________________________________
      public String getParameter(String name)

      { String result = _request.getParameter(name); if ( result != null ) return result; IMultipartDecoder decoder = getDecoder(); if (decoder != null) return decoder.getString(_request, name); return null; }

      ________________________________________

      Apply analogous patch to the method getParameters(String) in the same class.

        Activity

        Marcus Schulte created issue -
        Jesse Kuhnert made changes -
        Field Original Value New Value
        Fix Version/s 4.1.1 [ 12312021 ]
        Jesse Kuhnert made changes -
        Fix Version/s 4.1.1 [ 12312021 ]
        Fix Version/s 4.1.2 [ 12312202 ]
        Jesse Kuhnert made changes -
        Fix Version/s 4.1.2 [ 12312202 ]
        Fix Version/s 4.1.3 [ 12312516 ]
        Marcus Schulte made changes -
        Assignee Marcus Schulte [ mschu ]
        Marcus Schulte made changes -
        Resolution Fixed [ 1 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Mark Thomas made changes -
        Workflow jira [ 12321312 ] Default workflow, editable Closed status [ 12567036 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12567036 ] jira [ 12589782 ]

          People

          • Assignee:
            Marcus Schulte
            Reporter:
            Marcus Schulte
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development