search

albertosantini / node-rio

Integration with Rserve, a TCP/IP server for R framework
https://github.com/albertosantini/node-conpa
MIT License
176 stars 35 forks source link

Enable connection to Rserve over Websockets #24

Closed alexproca closed 9 years ago

alexproca commented 9 years ago

Since Rserve 1.7.3 the server can be accesed over Websockets. If this kind of connection would be possible from rio, the library will be usable from browser too. But this will be fesable after implementing 'Support crypted authentication'

albertosantini commented 9 years ago

Considered a few time ago.

I don't know if it would be a best practice, in term of architectural design, to connect directly the frontend (browser) layer with the math engine running in the backend layer.

I think there should be always something in the middle, at least for security and authentication concerns. If there is a middle tier, I suppose there is no need to add that transport also to rio.

However the main advantage of the Websockets transport is the out of band message. For instance, it would be possible to update the status on client side for a long running process.

Anyway that feature is a bit out of scope by design: request and reply approach is the main scenario, where the reply is below the second. If the user needs long processes control, it is a different use case.

albertosantini commented 9 years ago

sessions (reusable connections) and browserify (if implemented and if it works) should mitigate finally the needs underlying the use cases of this issue.

albertosantini commented 9 years ago

I would close this issue. See also #11.

Maybe we can reconsider it later, for dev-3.x.

alexproca commented 9 years ago

Ok On Dec 1, 2014 12:05 PM, "Alberto Santini" notifications@github.com wrote:

I would close this issue. See also #11 https://github.com/albertosantini/node-rio/issues/11.

Maybe we can reconsider it later, for dev-3.x.

— Reply to this email directly or view it on GitHub https://github.com/albertosantini/node-rio/issues/24#issuecomment-65043007 .