Skip to content

Commit 665681d

Browse files
authored
Merge pull request #667 from cloudflare/dominik/fix-sockets-errors-in-sentry
Reject closeFulfiller with jsg exception instead of KJ_EXCEPTION.
2 parents 88863c1 + 76d365b commit 665681d

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/workerd/api/sockets.c++

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -289,8 +289,7 @@ jsg::Promise<void> Socket::maybeCloseWriteSide(jsg::Lock& js) {
289289
// been flushed.
290290
return writable->getController().close(js).catch_(js,
291291
JSG_VISITABLE_LAMBDA((ref=JSG_THIS), (ref), (jsg::Lock& js, jsg::Value&& exc) {
292-
ref->closeFulfiller.resolver.reject(js,
293-
KJ_EXCEPTION(FAILED, kj::str(exc.getHandle(js.v8Isolate))));
292+
ref->closeFulfiller.resolver.reject(js, exc.getHandle(js.v8Isolate));
294293
})).then(js, JSG_VISITABLE_LAMBDA((ref=JSG_THIS), (ref), (jsg::Lock& js) {
295294
ref->closeFulfiller.resolver.resolve();
296295
}));

0 commit comments

Comments
 (0)