[ 
https://issues.apache.org/jira/browse/THRIFT-2355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13922337#comment-13922337
 ] 

Henrique Mendonça commented on THRIFT-2355:
-------------------------------------------

Hi Randy,

Thanks for clarifying.
That's totally fine, we are anyway free to implement any extension to the 
library but it should be clear to the community why certain decisions were 
made. I like your implementation (apart perhaps for using decimals to index 
bits, Lol) and it seams to be neat and clear enough for further reuse, and we 
still stay open for other implementations if it makes sense. As you has 
probably seen on the other thread, my dream was to get socket.io to run with 
thrift, as it also supports many target languages, but I never got time to 
really test it. Perhaps one day after they release 1.0 :)
Generally, I totally with you about [ not ] adding new dependencies to Thrift. 
However, I see it differently in the case of Node and NPM, and autoconf can be 
used to handle optional dependencies for other languages.

Cheers,
Henrique

> Add SSL and Web Socket Support to Node and JavaScript
> -----------------------------------------------------
>
>                 Key: THRIFT-2355
>                 URL: https://issues.apache.org/jira/browse/THRIFT-2355
>             Project: Thrift
>          Issue Type: Improvement
>          Components: JavaScript - Compiler, JavaScript - Library, Node.js - 
> Library
>    Affects Versions: 0.9.2
>         Environment: all
>            Reporter: Randy Abernethy
>            Assignee: Randy Abernethy
>            Priority: Minor
>             Fix For: 0.9.2
>
>         Attachments: 0001-Add-WebSocket-support-to-JS-and-Node.JS.patch
>
>
> This patch adds support for SSL between Node.js and Browser JavaScript as 
> well as initial WebSocket  transport support.
> The createStaticHTTPThriftServer() method has been renamed createWebServer(). 
> The resulting server supports XHR based HTTP:// and HTTPS:// clients as well 
> as upgrades to WS:// and WSS:// for websocket clients. 
> The JavaScript thrift.js library now includes websocket support through 
> TWebSocketTransport. The old XHR transport still goes by the alias Transport 
> but can now also be accessed through the name TXHRTransport. Browser clients 
> can use XHR over HTTP and HTTPS (as always) and also WebSocket (WS://) and 
> secure WebSocket (WSS://).
> The JavaScript grunt build runs jQuery and Normal JavaScript synchronous and 
> asynchronous tests over HTTP and HTTPS. There are now three test drivers in 
> the thrift/lib/js/test directory:
>   - test.html   the jQuery build test (thrift -gen js:jquery)
>   - test-nojq.html   the normal JavaScript build test (thrift -gen js)
>   - testws.html   the web socket test.
> The thrift/test/keys directory supplies the key and cert for the HTTPS test 
> server. 
> The root js README was updated and contains a complete client/server example. 
> A new README was added to the test directory providing an orientation there. 
> With WebSocket in place it makes sense to consider adding a binary protocol 
> on the browser side.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to