Igor Tkach created THRIFT-2995:
----------------------------------

             Summary: nodejs: index.js requires too much
                 Key: THRIFT-2995
                 URL: https://issues.apache.org/jira/browse/THRIFT-2995
             Project: Thrift
          Issue Type: Bug
          Components: Node.js - Library
    Affects Versions: 0.9.2
         Environment: webpack
            Reporter: Igor Tkach


THRIFT-2387 requested centralizing external exports in index.js. Required 
modules that are now in  index.js create unnecessary and problematic 
dependencies  which make it impossible to use thrift module in browser via 
tools like webpack. Client code may avoid using {{require('thrift')}} import 
like so:

{{
var options = {
    transport: require('thrift/transport').TBufferedTransport,
    protocol: require('thrift/protocol').TBinaryProtocol,
    path: "/",
    headers: {"Connection": "close"},
    https: false,
    responseType: 'arraybuffer'
  };
  var httpConnection = require('thrift/http_connection');
  var con = httpConnection.createHttpConnection("localhost", 8888, options);
  var client = httpConnection.createHttpClient(MyService, con);
}}   

but unfortunately generated {{MyService}} nodejs module contains  
{{require('thrift')}} which tries to bring in connection, ws_connection, server 
and web_server all of which of course are unwanted in the browser and depend on 
node modules that can't work in the browser.
  





--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to