Tapestry 5
  1. Tapestry 5
  2. TAP5-1900

Tapestry always uses application charset for request encoding, should first check content-type header

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 5.3.2, 5.3.3, 5.3.4, 5.3.5, 5.3.6
    • Fix Version/s: None
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      I'm using Tapestry as a sort of REST server, using pages to act as REST gateways to business services.

      To access those pages I'm using Spring's RestTemplate which as a default uses ISO-8859-1 as the request encoding and properly seting the request's content-type header.

      Tapestry currently forces the request encoding to be the same as the application charset (see org.apache.tapestry5.internal.services.RequestImpl.setupEncoding), I believe it should first check the content-type header and use that value instead, only using application charset as a fallback.

      1. tap5-1900.patch
        1 kB
        Nelson Rodrigues

        Activity

        Nelson Rodrigues created issue -
        Nelson Rodrigues made changes -
        Field Original Value New Value
        Attachment tap5-1900.patch [ 12522583 ]
        Lance made changes -
        Link This issue relates to TAP5-1741 [ TAP5-1741 ]
        Lance made changes -
        Link This issue relates to TAP5-1741 [ TAP5-1741 ]
        Nelson Rodrigues made changes -
        Affects Version/s 5.3.3 [ 12320045 ]
        Lance made changes -
        Link This issue is related to TAP5-1741 [ TAP5-1741 ]
        Lance made changes -
        Link This issue is related to TAP5-1741 [ TAP5-1741 ]
        Nelson Rodrigues made changes -
        Affects Version/s 5.3.6 [ 12322961 ]
        Affects Version/s 5.3.5 [ 12322440 ]
        Affects Version/s 5.3.4 [ 12320774 ]
        Howard M. Lewis Ship made changes -
        Assignee Howard M. Lewis Ship [ hlship ]

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Nelson Rodrigues
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:

              Development