You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Hudson (JIRA)" <ji...@apache.org> on 2015/02/02 14:46:34 UTC
[jira] [Commented] (THRIFT-2934) createServer signature mismatch
[ https://issues.apache.org/jira/browse/THRIFT-2934?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14301278#comment-14301278 ]
Hudson commented on THRIFT-2934:
--------------------------------
SUCCESS: Integrated in Thrift #1438 (See [https://builds.apache.org/job/Thrift/1438/])
THRIFT-2934: make node server options optional (ra: rev 1097c40ed0f36f44081d454dab7914673fb6ea07)
* lib/nodejs/lib/thrift/server.js
> createServer signature mismatch
> -------------------------------
>
> Key: THRIFT-2934
> URL: https://issues.apache.org/jira/browse/THRIFT-2934
> Project: Thrift
> Issue Type: Bug
> Components: Node.js - Library, Tutorial
> Affects Versions: 0.9.2
> Reporter: Alan Cheung
> Assignee: Randy Abernethy
> Priority: Trivial
> Fix For: 0.9.3
>
> Attachments: 0001-make-options-optional-again-in-node-server.patch
>
>
> The nodejs example https://thrift.apache.org/tutorial/nodejs
> If we createServer without a third option argument, would result in
> if (options.tls) {
> ^
> TypeError: Cannot read property 'tls' of undefined
> at Object.exports.createMultiplexServer (...\ProjRoot\node_modules\thrift\lib\thrift\s
> erver.js:89:14)
> at Object.exports.createServer (...\ProjRoot\node_modules\thrift\lib\thrift\server.js:106:18)
> at Object.<anonymous> (...\ProjRoot\NodeServer.js:8:21)
> at Module._compile (module.js:456:26)
> at Object.Module._extensions..js (module.js:474:10)
> at Module.load (module.js:356:32)
> at Function.Module._load (module.js:312:12)
> at Function.Module.runMain (module.js:497:10)
> at startup (node.js:119:16)
> at node.js:929:3
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)