cohabit/website
7
0
Fork 0
Code Issues 13 Pull requests Actions Packages Projects Releases Wiki Activity

fix(island): 🐛 multiple intervals are started at same time checkin is-online

Browse source
This commit is contained in:
Julien Oculi 2024-07-19 15:24:14 +02:00
parent 82906907e2
commit 470467d8a8
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
islands/IsOnline.tsx
View file

@ -32,7 +32,7 @@ export default function IsOnline(
? navigator.connection as NetworkConnection
: null
useEffect(() => {
openSocket(status, { id: undefined })
openSocket(status)
connection?.addEventListener('change', () => {
fetch('/api/serviceworker/is-online')
.then(() => status.value = true)
@ -43,18 +43,19 @@ export default function IsOnline(
return <>{displayed}</>
}
function openSocket(status: Signal<boolean>, ref: { id: number | undefined }) {
function openSocket(status: Signal<boolean>) {
const socket = new WebSocket(
`wss://${location.host}/api/serviceworker/is-online`,
)
socket.addEventListener('open', () => {
status.value = true
clearInterval(ref.id)
const id = Number(sessionStorage.getItem('$cohabit.is-online.id') ?? -1)
clearInterval(id)
})
socket.addEventListener('close', () => {
status.value = false
// Try reconnect every 5s
const id = setInterval(() => openSocket(status, ref), 5_000)
ref.id = id
const id = setInterval(() => openSocket(status), 5_000)
sessionStorage.setItem('$cohabit.is-online.id', id.toString())
})
}