Struts 2
  1. Struts 2
  2. WW-3752

Charset encoding incorrect when using parameters in ServletRedirectResult

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Not a Problem
    • Affects Version/s: 2.3.1.1
    • Fix Version/s: 2.3.15
    • Component/s: Dispatch Filter
    • Labels:
      None
    • Environment:

      Tomcat 6

      Description

      See attached test case.

      The issue seems to be that when using some parameters with a ServletRedirectResult, they get encoded twice in UTF-8 : once in UrlHelper.buildParametersString and a second time in response.encodeRedirectURL.

      1. WW-3752.zip
        20 kB
        Denis Cabasson

        Activity

        Hide
        Denis Cabasson added a comment - - edited

        a test case for the issue, based on the struts2-archetype-blank. Just go to http://localhost:8080/WW-3752/example/HelloWorld and follow the 2 links at the bottom. The first one with ASCII parameters works fine, the second not.

        Show
        Denis Cabasson added a comment - - edited a test case for the issue, based on the struts2-archetype-blank. Just go to http://localhost:8080/WW-3752/example/HelloWorld and follow the 2 links at the bottom. The first one with ASCII parameters works fine, the second not.
        Hide
        Denis Cabasson added a comment -

        Apparently, the bug occurs in Tomcat 6.0.32 (which I am using), but not in jetty 6.1.21. I will upgrade my Tomcat and see if the issue is still there in the latest version of Tomcat.

        Show
        Denis Cabasson added a comment - Apparently, the bug occurs in Tomcat 6.0.32 (which I am using), but not in jetty 6.1.21. I will upgrade my Tomcat and see if the issue is still there in the latest version of Tomcat.
        Hide
        Lukasz Lenart added a comment -

        Denis Cabasson any news ?

        Show
        Lukasz Lenart added a comment - Denis Cabasson any news ?
        Hide
        Lukasz Lenart added a comment -

        Denis Cabasson please reopen if you disagree

        Show
        Lukasz Lenart added a comment - Denis Cabasson please reopen if you disagree

          People

          • Assignee:
            Lukasz Lenart
            Reporter:
            Denis Cabasson
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development