[ 
https://issues.apache.org/jira/browse/THRIFT-3323?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14935715#comment-14935715
 ] 

ASF GitHub Bot commented on THRIFT-3323:
----------------------------------------

Github user Jens-G commented on the pull request:

    https://github.com/apache/thrift/pull/605#issuecomment-144166076
  
    Committed per 15c400488550d565e5436f1e77ad3fdddfd4ef66
    Could you pleaswe close the PR manually? Thanks!


> Python library does not handle escaped forward slash ("/") in JSON
> ------------------------------------------------------------------
>
>                 Key: THRIFT-3323
>                 URL: https://issues.apache.org/jira/browse/THRIFT-3323
>             Project: Thrift
>          Issue Type: Bug
>          Components: Python - Library
>    Affects Versions: 0.9.2
>            Reporter: Subrat Basnet
>            Assignee: Jens Geyer
>            Priority: Minor
>             Fix For: 0.9.4
>
>
> As per json.org - a slash ("/") may be escaped too.
> When parsing a JSON that has escaped slashes, the Python library throws an 
> exception as follows:
> Traceback (most recent call last):
>   File "lib/crawldata.py", line 32, in __init__
>     deserialize(base=self, buf=thrift_buf, 
> protocol_factory=CrawlDataTJSONProtocolFactory())
>   File "/usr/local/lib/python2.7/dist-packages/thrift/TSerialization.py", 
> line 37, in deserialize
>     base.read(protocol)
>   File "models/thrift/crawldata/ttypes.py", line 761, in read
>     self.data.read(iprot)
>   File "models/thrift/crawldata/ttypes.py", line 241, in read
>     _val19 = iprot.readString();
>   File 
> "/usr/local/lib/python2.7/dist-packages/thrift/protocol/TJSONProtocol.py", 
> line 390, in readString
>     return self.readJSONString(False)
>   File 
> "/usr/local/lib/python2.7/dist-packages/thrift/protocol/TJSONProtocol.py", 
> line 235, in readJSONString
>     "Expected control char")
> TProtocolException: Expected control char



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to