public/browser
1
0
Fork 0
mirror of https://github.com/lightpanda-io/browser.git synced 2026-03-22 12:44:43 +00:00
Code Issues Actions 5 Packages Projects Releases Wiki Activity

Don't log SocketNotConnected when shutting down listener on non-Linux

Browse Source 
On BSD, a listening socket isn't considered connected, so this error is
expected. Maybe we shouldn't call shutdown at all, but I guess it's safer this
way.
This commit is contained in:
Karl Seguin
2026-03-11 16:29:44 +08:00
parent a725e2aa6a
commit 48ead90850
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
src/network/Runtime.zig
View File

@@ -180,7 +180,13 @@ pub fn run(self: *Runtime) void {
}
if (self.listener) |listener| {
posix.shutdown(listener.socket, .both) catch |err| {
posix.shutdown(listener.socket, .both) catch |err| blk: {
if (err == error.SocketNotConnected and builtin.os.tag != .linux) {
// This error is normal/expected on BSD/MacOS. We probably
// shouldn't bother calling shutdown at all, but I guess this
// is safer.
break :blk;
}
lp.log.warn(.app, "listener shutdown", .{ .err = err });
};
posix.close(listener.socket);