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

Node.js server does not fully support exception

    XMLWordPrintableJSON

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.3
    • Component/s: Node.js - Compiler
    • Labels:
      None
    • Patch Info:
      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

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

              Dates

              • Created:
                Updated:
                Resolved: