[
https://issues.apache.org/jira/browse/THRIFT-3746?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15201599#comment-15201599
]
ASF GitHub Bot commented on THRIFT-3746:
----------------------------------------
GitHub user tylertreat-wf opened a pull request:
https://github.com/apache/thrift/pull/955
THRIFT-3746 JSON protocol left in incorrect state on read errors
This is related to THRIFT-3735, but the problem was not fully addressed
there. The JSON protocol's read buffer needs to be reset to avoid being
put in an invalid state on read errors.
This also changes how the writer is reset to match what we do with the
reader for consistency.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tylertreat-wf/thrift THRIFT-3746
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/thrift/pull/955.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #955
----
commit 7503e55f5cc3ed65e80cf0df1da9e76ba900e103
Author: Tyler Treat <[email protected]>
Date: 2016-03-16T15:06:55Z
THRIFT-3746 JSON protocol left in incorrect state on read errors
This is related to THRIFT-3735, but the problem was not fully addressed
there. The JSON protocol's read buffer needs to be reset to avoid being
put in an invalid state on read errors.
This also changes how the writer is reset to match what we do with the
reader for consistency.
----
> JSON protocol left in incorrect state on read errors
> ----------------------------------------------------
>
> Key: THRIFT-3746
> URL: https://issues.apache.org/jira/browse/THRIFT-3746
> Project: Thrift
> Issue Type: Bug
> Components: Go - Library
> Reporter: Tyler Treat
>
> This is related to THRIFT-3735, but the problem was not fully addressed
> there. The JSON protocol's read buffer needs to be reset to avoid being put
> in an invalid state on read errors.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
