Socket close handler not working in browser setting

[felixSchl]

Hey, I noticed the socket close handler is not working in the browser

js-libp2p-websockets/src/socket-to-conn.ts

Lines 61 to 68 in ac069d3

	stream.socket.once != null && stream.socket.once('close', () => { // eslint-disable-line @typescript-eslint/prefer-optional-chain
	// In instances where `close` was not explicitly called,
	// such as an iterable stream ending, ensure we have set the close
	// timeline
	if (maConn.timeline.close == null) {
	maConn.timeline.close = Date.now()
	}
	})

. To verify I have locally edited socket-to-conn.js and replaced the .once stuff with this:

function onclose () {
    if (stream.socket.onclose === onclose) {
        stream.socket.onclose = null; // only once
    }
    if (maConn.timeline.close == null) {
        maConn.timeline.close = Date.now();
    }
}
stream.socket.onclose = onclose;

It has the downside of mutating the object so there might be a better solution, but it works.

[github-actions]

🎉 This issue has been resolved in version 3.0.4 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀