Skip to content

Commit

Permalink
Merge pull request #85 from gadget-inc/nicholas/catch-unexpected-epipes
Browse files Browse the repository at this point in the history
Added request error handling to notification flush
  • Loading branch information
pistachiobaby authored Jul 19, 2023
2 parents bd24f6d + e2da2cc commit abf9ecf
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "wds",
"version": "0.17.1",
"version": "0.17.2",
"author": "Harry Brundage",
"license": "MIT",
"bin": {
Expand Down
3 changes: 3 additions & 0 deletions src/child-process-registration.ts
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,9 @@ const throttledRequireFlush = throttle(() => {
300
);

request.on("error", (error: any) => {
log.debug(`Unexpected request error while flushing require notifications`, error);
});
request.write(JSON.stringify(pendingRequireNotifications));
request.end();
pendingRequireNotifications = [];
Expand Down

0 comments on commit abf9ecf

Please sign in to comment.