Struts 2
  1. Struts 2
  2. WW-3945

Set default encoding for JSONResult to UTF-8

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.3.7
    • Fix Version/s: 2.5
    • Component/s: None
    • Labels:
      None

      Description

      Check WW-3863 for more details, but encoding param must be set to UTF-8 by default

        Activity

        Hide
        victorsosa added a comment -

        check PR# 66

        Show
        victorsosa added a comment - check PR# 66
        Hide
        ASF GitHub Bot added a comment -

        Github user lukaszlenart commented on the pull request:

        https://github.com/apache/struts/pull/66#issuecomment-168777661

        Conflicts

        Show
        ASF GitHub Bot added a comment - Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/66#issuecomment-168777661 Conflicts
        Hide
        ASF GitHub Bot added a comment -

        Github user victorsosa commented on the pull request:

        https://github.com/apache/struts/pull/66#issuecomment-168777910

        Yea, My bad; Let me fix it

        Show
        ASF GitHub Bot added a comment - Github user victorsosa commented on the pull request: https://github.com/apache/struts/pull/66#issuecomment-168777910 Yea, My bad; Let me fix it
        Hide
        ASF GitHub Bot added a comment -

        Github user victorsosa commented on the pull request:

        https://github.com/apache/struts/pull/66#issuecomment-168781813

        Done, check Jenkins; it seems to be blocked

        Show
        ASF GitHub Bot added a comment - Github user victorsosa commented on the pull request: https://github.com/apache/struts/pull/66#issuecomment-168781813 Done, check Jenkins; it seems to be blocked
        Hide
        ASF GitHub Bot added a comment -

        Github user lukaszlenart commented on the pull request:

        https://github.com/apache/struts/pull/66#issuecomment-168782251

        Yes, there is few other builds in pipeline

        Show
        ASF GitHub Bot added a comment - Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/66#issuecomment-168782251 Yes, there is few other builds in pipeline
        Hide
        Lukasz Lenart added a comment -

        PR merged, thanks!

        Show
        Lukasz Lenart added a comment - PR merged, thanks!
        Hide
        ASF subversion and git services added a comment -

        Commit f9b7ebb87eb2beeffb2ca1b0f3b421b0cef4d658 in struts's branch refs/heads/master from Victor Sosa
        [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=f9b7ebb ]

        Patch WW-3945

        Set default encoding for JSONResult to UTF-8

        Now the default encoding is UTF-8 for JSON as should be.

        Show
        ASF subversion and git services added a comment - Commit f9b7ebb87eb2beeffb2ca1b0f3b421b0cef4d658 in struts's branch refs/heads/master from Victor Sosa [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=f9b7ebb ] Patch WW-3945 Set default encoding for JSONResult to UTF-8 Now the default encoding is UTF-8 for JSON as should be.
        Hide
        ASF subversion and git services added a comment -

        Commit b198e3bf3d70f7bfd5b8d560a848261b4855be2e in struts's branch refs/heads/master from Victor Sosa
        [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=b198e3b ]

        Patch WW-3945

        Set default encoding for JSONResult to UTF-8

        Now the default encoding is UTF-8 for JSON as should be.

        Show
        ASF subversion and git services added a comment - Commit b198e3bf3d70f7bfd5b8d560a848261b4855be2e in struts's branch refs/heads/master from Victor Sosa [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=b198e3b ] Patch WW-3945 Set default encoding for JSONResult to UTF-8 Now the default encoding is UTF-8 for JSON as should be.
        Hide
        ASF subversion and git services added a comment -

        Commit acd55986f4d860c55efb8fae56ec5344d5b9780a in struts's branch refs/heads/master from Lukasz Lenart
        [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=acd5598 ]

        WW-3945 Sets encoding in JSON result to UTF-8 by default

        Show
        ASF subversion and git services added a comment - Commit acd55986f4d860c55efb8fae56ec5344d5b9780a in struts's branch refs/heads/master from Lukasz Lenart [ https://git-wip-us.apache.org/repos/asf?p=struts.git;h=acd5598 ] WW-3945 Sets encoding in JSON result to UTF-8 by default
        Hide
        ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/struts/pull/66

        Show
        ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/struts/pull/66
        Hide
        ASF GitHub Bot added a comment -

        Github user victorsosa commented on the pull request:

        https://github.com/apache/struts/pull/66#issuecomment-168793429

        :+1:

        Show
        ASF GitHub Bot added a comment - Github user victorsosa commented on the pull request: https://github.com/apache/struts/pull/66#issuecomment-168793429 :+1:
        Hide
        Hudson added a comment -

        SUCCESS: Integrated in Struts-JDK7-master #396 (See https://builds.apache.org/job/Struts-JDK7-master/396/)
        Patch WW-3945 (victornsosa: rev f9b7ebb87eb2beeffb2ca1b0f3b421b0cef4d658)

        • plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java
        • plugins/json/src/main/java/org/apache/struts2/json/JSONResult.java
          Patch WW-3945 (victornsosa: rev b198e3bf3d70f7bfd5b8d560a848261b4855be2e)
        • plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java
        • plugins/json/src/main/java/org/apache/struts2/json/JSONResult.java
        Show
        Hudson added a comment - SUCCESS: Integrated in Struts-JDK7-master #396 (See https://builds.apache.org/job/Struts-JDK7-master/396/ ) Patch WW-3945 (victornsosa: rev f9b7ebb87eb2beeffb2ca1b0f3b421b0cef4d658) plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java plugins/json/src/main/java/org/apache/struts2/json/JSONResult.java Patch WW-3945 (victornsosa: rev b198e3bf3d70f7bfd5b8d560a848261b4855be2e) plugins/jasperreports/src/main/java/org/apache/struts2/views/jasperreports/JasperReportsResult.java plugins/json/src/main/java/org/apache/struts2/json/JSONResult.java

          People

          • Assignee:
            Unassigned
            Reporter:
            Lukasz Lenart
          • Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development