search

jbmusso / gremlin-javascript

JavaScript tools for graph processing in Node.js and the browser inspired by the Apache TinkerPop API
MIT License
214 stars 63 forks source link

Allow user to close the connection #81

Open brianfoody opened 7 years ago

brianfoody commented 7 years ago

I run this in an AWS lambda environment and need the ability to end the connection. It's appears to be missing so I have added it to WebSocketGremlinConnection.js.

jbmusso commented 7 years ago

Thanks, on my phone right now - I will review tomorrow.

On Fri, 14 Jul 2017 at 14:33, Brian notifications@github.com wrote:

I run this in an AWS lambda environment and need the ability to end the connection. It's appears to be missing so I have added it to WebSocketGremlinConnection.js.

You can view, comment on, or merge this pull request online at:

https://github.com/jbmusso/gremlin-javascript/pull/81 Commit Summary

  • Add ability to terminate connection
  • Add ability to terminate connection
  • Delete .DS_Store

File Changes

Patch Links:

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/jbmusso/gremlin-javascript/pull/81, or mute the thread https://github.com/notifications/unsubscribe-auth/AAMa1AyapXX_Bjsc65t02c2TdRXW6iWYks5sN2AFgaJpZM4OYMsv .

brianfoody commented 7 years ago

Hey @jbmusso . Any chance of merging this and releasing a new npm version?