Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Bugfix/close open files on disconnect #77

Merged
merged 6 commits into from Dec 14, 2018

Conversation

stephen-palmer
Copy link
Contributor

Fix Issue #69: ensure that open file streams are closed if the socket is destroyed
Fix Issue #68: fixed infinite setTimeout loop when socket is destroyed during a file read

… is destroyed.

Fix Issue #68: fixed infinite setTimeout loop when socket is destroyed during a file read
Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ A review job has been created and sent to the PullRequest network.


Check the status or cancel PullRequest code review here - or - cancel by adding [!pr] to the title of the pull request.

@coveralls
Copy link

coveralls commented Dec 14, 2018

Coverage Status

Coverage increased (+0.4%) to 92.837% when pulling 0aa8456 on bugfix/close-open-files-on-disconnect into 0b83660 on master.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✖️ This code review was cancelled. See Details

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ A review job has been created and sent to the PullRequest network.


Check the status or cancel PullRequest code review here - or - cancel by adding [!pr] to the title of the pull request.

Copy link

@pullrequest pullrequest bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this is looking pretty good and leaves the code in an improved state, nice work!


Was this helpful? Yes | No

test/protocol.js Show resolved Hide resolved
lib/server/command_processor.js Outdated Show resolved Hide resolved

stream.on('readable', readChunk);
const endRead = () => {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice job putting this in a shared handler instead of duplicating code.

test/protocol.js Show resolved Hide resolved
test/test_utils.js Outdated Show resolved Hide resolved
@stephen-palmer stephen-palmer merged commit 91382cf into master Dec 14, 2018
@stephen-palmer stephen-palmer deleted the bugfix/close-open-files-on-disconnect branch December 14, 2018 16:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants