amark / gun

An open source cybersecurity protocol for syncing decentralized graph data.
https://gun.eco/docs
Other
17.93k stars 1.16k forks source link

WS ws.path fix #1343

Closed bmatusiak closed 7 months ago

bmatusiak commented 7 months ago

i found that i always run into issues with it processing the correct path.

so this should fix gun when using WebSockets with other ws protocols that attach to different paths (webpack , socket.io)

bmatusiak commented 7 months ago

cleaned up, ready to review @amark

amark commented 7 months ago

@bmatusiak cool, it seems like some || edge case? I don't quite understand but you seem to notice it fixes stuff & I appreciate that + trust you! You're awesome, an open source hero!!!!

bmatusiak commented 7 months ago

thanks! || prevents overloading it i guess. there was a edge case in the past from a comment i seen.. but i think what i did was the original intent

bmatusiak commented 7 months ago

i think the edge was.. if you want to host gun.js but not use the websockets