Thrift
  1. Thrift
  2. THRIFT-1131

C# JSON Protocol is unable to decode escaped characters in string

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.6
    • Fix Version/s: 0.6.1, 0.7
    • Component/s: C# - Library
    • Labels:
      None
    • Environment:

      C#, Windows 7, VS 2010

    • Patch Info:
      Patch Available

      Description

      C# JSON Protocol throws TProtocolException after meeting an escaped character in UTF8 string. I attach a patch.

      1. TJSONProtocol.patch
        0.5 kB
        Maciek Weksej

        Activity

        Hide
        Hudson added a comment -

        Integrated in Thrift #128 (See https://hudson.apache.org/hudson/job/Thrift/128/)

        Show
        Hudson added a comment - Integrated in Thrift #128 (See https://hudson.apache.org/hudson/job/Thrift/128/ )
        Bryan Duxbury made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Bryan Duxbury made changes -
        Fix Version/s 0.7 [ 12315360 ]
        Jake Farrell made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Maciek Weksej [ katoda ]
        Resolution Fixed [ 1 ]
        Hide
        Jake Farrell added a comment -

        Applied to 0.6.x branch and trunk, thanks for the patch

        Show
        Jake Farrell added a comment - Applied to 0.6.x branch and trunk, thanks for the patch
        Maciek Weksej made changes -
        Priority Minor [ 4 ] Major [ 3 ]
        Maciek Weksej made changes -
        Environment C#, Windows 7 C#, Windows 7, VS 2010
        Maciek Weksej made changes -
        Comment [ Simple patch ]
        Maciek Weksej made changes -
        Field Original Value New Value
        Attachment TJSONProtocol.patch [ 12475800 ]
        Maciek Weksej created issue -

          People

          • Assignee:
            Maciek Weksej
            Reporter:
            Maciek Weksej
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development