Wicket
  1. Wicket
  2. WICKET-1388

"charset=UTF-8" is appended to Content-Type of DynamicWebResources

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.1
    • Fix Version/s: 1.3.2
    • Component/s: wicket
    • Labels:
      None
    • Environment:
      IE 6.0, Acrobat Reader 7.0.9

      Description

      After upgrading from wicket 1.2.6 to 1.3.1 we now have a big problem when opening PDF files inside the IE browser window.
      Instead the PDF file is now loaded as a separate window, leaving the Browser window empty.

      I traced the problem down wicket always appending ";charset=UTF-8" to the Content-Type, and this can be viewed by using a http debugger:

      Wicket 1.2.6: Works
      Content-Type: application/pdf

      Wicket 1.3.1: PDF opened outside the browser.
      Content-Type: application/pdf; charset=UTF-8

      The ";charset=UTF-8" is appended in ResourceStreamRequestTarget, and I can't see a way to override it.

      Please make it possible to have the Content-Type as set by the DynamicWebResource

        Activity

        Hide
        Johan Compagner added a comment -

        this is weird, ResourceStreamRequestTarget does set that charset already long long time.
        But there are other changes in that area.

        i guess it only makes sense to append charset=XXX to the content type if the type is of type text so i changed the code to check first the content type

        Show
        Johan Compagner added a comment - this is weird, ResourceStreamRequestTarget does set that charset already long long time. But there are other changes in that area. i guess it only makes sense to append charset=XXX to the content type if the type is of type text so i changed the code to check first the content type

          People

          • Assignee:
            Johan Compagner
            Reporter:
            Niels Bo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development