Struts 2
  1. Struts 2
  2. WW-4023

ParametersInterceptor produces a warning when parameter method:* is set

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.7, 2.3.8, 2.3.12
    • Fix Version/s: 2.3.16
    • Component/s: Core Interceptors
    • Labels:
      None

      Description

      The following example produces a form with the parameter names: number and method:add

      <s:form namespace="/example" action="example">
      <s:textfield name="number"/>
      <s:submit method="add" value="Add"></s:submit>
      </s:form>
      

      The ParametersInterceptor throws a warning when devMode is set:

      Parameter [method:add] didn't match acceptedPattern pattern!

      I think this warn message shouldn't be logged because it's a normal use case. The "method:" parameter should be included in the excludeParams configuration.

      This issue relates to WW-3973.

        Issue Links

          Activity

          Jon Y. created issue -
          Lukasz Lenart made changes -
          Field Original Value New Value
          Fix Version/s 2.3.14 [ 12324256 ]
          Lukasz Lenart made changes -
          Fix Version/s 2.3.15 [ 12324267 ]
          Fix Version/s 2.3.14 [ 12324256 ]
          Lukasz Lenart made changes -
          Priority Major [ 3 ] Minor [ 4 ]
          Lukasz Lenart made changes -
          Fix Version/s 2.3.16 [ 12324546 ]
          Fix Version/s 2.3.15 [ 12324267 ]
          Lukasz Lenart made changes -
          Fix Version/s 2.3.17 [ 12324780 ]
          Fix Version/s 2.3.16 [ 12324546 ]
          Lukasz Lenart made changes -
          Fix Version/s 2.3.16 [ 12324546 ]
          Fix Version/s 2.3.17 [ 12324780 ]
          Lukasz Lenart made changes -
          Link This issue duplicates WW-4132 [ WW-4132 ]
          Lukasz Lenart made changes -
          Comment [ Problem occurs only in devMode. The second is that ParametersInterceptor is in XWork core, but method:, action: and redirect: are defined in DefaultActionMapper in Struts2 core :\

          Adding them as a excludeParams will add cross dependency, I don't see a good solution now. ]
          Lukasz Lenart made changes -
          Assignee Lukasz Lenart [ lukaszlenart ]
          Lukasz Lenart made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Lukasz Lenart made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Lukasz Lenart
              Reporter:
              Jon Y.
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development