Uploaded image for project: 'Thrift'
  1. Thrift
  2. THRIFT-3008

Node.js server does not fully support exception

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.9.3
    • Node.js - Compiler
    • None
    • Patch Available

    Description

      To my understanding (based on comment in TestThrift.thrift and C++ server behavior), servers should be able to send T(Application)Exception when an error occurred in the handler.

      Node.js server supports exceptions explicitly declared with "throws", but does not support the type of exceptions stated above.
      Also, the client test (lib/nodejs/test/test_driver.js) assumes current Node.js server behavior, resulting in cross test failures against C++ and Java reference implementations.

      This patch makes compiler generate server code that can emit errors in an identical way to C++ server code.

      Attachments

        Activity

          People

            codesf Randy Abernethy
            nsuke Nobuaki Sukegawa
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: