You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "James E. King, III (JIRA)" <ji...@apache.org> on 2017/03/30 15:50:41 UTC
[jira] [Updated] (THRIFT-4150) Python server or cpp client do not
handle an unimplemented service method gracefully
[ https://issues.apache.org/jira/browse/THRIFT-4150?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
James E. King, III updated THRIFT-4150:
---------------------------------------
Summary: Python server or cpp client do not handle an unimplemented service method gracefully (was: Python server or cpp client do not handle the unimplemented method gracefully)
> Python server or cpp client do not handle an unimplemented service method gracefully
> ------------------------------------------------------------------------------------
>
> Key: THRIFT-4150
> URL: https://issues.apache.org/jira/browse/THRIFT-4150
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Compiler, C++ - Library, Python - Compiler, Python - Library
> Affects Versions: 0.10.0
> Environment: Ubuntu Docker
> Reporter: James E. King, III
> Priority: Minor
>
> I added a new method to the ThriftTest service and I implemented the server and client in C++. In the C++ client I added exception handling to catch TProtocolException and std::exception. I expected that when TestClient sent in a method that the server had not implemented, a TProtocolException(NOT_IMPLEMENTED) would be returned. The server log indicates it got to this point in theory:
> {noformat}
> ERROR:root:'TestHandler' object has no attribute 'testRequired'
> Traceback (most recent call last):
> File "/home/jking/thrift/github/thrift/test/py/gen-py/ThriftTest/ThriftTest.py", line 1593, in process_testRequired
> self._handler.testRequired(args.req)
> {noformat}
> However on the C++ client side, a std::exception was received and NOT a TProtocolException, which is a bug:
> {noformat}
> testClient.testRequired() =>*** FAILED *** got back std::exception Internal error
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)