Wink
  1. Wink
  2. WINK-169 Cleanup Atom related providers
  3. WINK-171

JAXBElementXmlProvider ignores charset during unmarsheling

    Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.1
    • Fix Version/s: 1.0
    • Component/s: Common
    • Labels:
      None

      Description

      JAXBElementXmlProvider uses StreamSource during unmarsheling
      so it actually ignores the charset if it was set on MediaType.

      Should use Reader instead.

        Activity

        Michael Elman created issue -
        Michael Elman made changes -
        Field Original Value New Value
        Fix Version/s 0.2 [ 12314064 ]
        Affects Version/s 0.1 [ 12314063 ]
        Component/s Common [ 12312932 ]
        Michael Elman made changes -
        Description JAXBElementXmlProvider uses StreamSource during unmarsheling
        so it actually ignores the charset if it was set on MediaType.

        Should you Reader instead.
        JAXBElementXmlProvider uses StreamSource during unmarsheling
        so it actually ignores the charset if it was set on MediaType.

        Should use Reader instead.
        Hide
        Nick Gallardo added a comment -

        I'm away and not able to do it now, but I'd like to get a snapshot of what the performance difference is between using the StreamSource and the Reader. If the difference is noticeable, would you mind if we only used the Reader when the charset is actually present?

        Show
        Nick Gallardo added a comment - I'm away and not able to do it now, but I'd like to get a snapshot of what the performance difference is between using the StreamSource and the Reader. If the difference is noticeable, would you mind if we only used the Reader when the charset is actually present?
        Hide
        Michael Elman added a comment -

        only used the Reader when the charset is actually present

        I'll implement it this way.
        It can be always changed later.

        Show
        Michael Elman added a comment - only used the Reader when the charset is actually present I'll implement it this way. It can be always changed later.
        Michael Elman made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Michael Elman [ elman ]
        Resolution Fixed [ 1 ]
        Michael Elman made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Michael Elman
            Reporter:
            Michael Elman
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development