internetarchive / dweb-mirror

Offline Internet Archive project
https://www-dweb-mirror.dev.archive.org/
GNU Affero General Public License v3.0
263 stars 27 forks source link

Online status isn't getting updated correctly #237

Closed JesseWeinstein closed 5 years ago

JesseWeinstein commented 5 years ago

When I go offline, the circle doesn't turn red, even the the console shows failing online checks. I'll investigate more.

mitra42 commented 5 years ago
mitra42 commented 5 years ago

RESEARCH Core function is ReactSupport.transportStatusAndProps This is called from Page.constructor -

mitra42 commented 5 years ago

CLEAN UP (after bug fixed)

mitra42 commented 5 years ago

DIAGNOSIS Running with WiFi on - so should show online (its not)

mitra42 commented 5 years ago

Obscure ! It was triggering a process called "loopguard" which tries to remove queued http fetches when a page switches. This should have resulted in an error, but the error was being discarded in httptools.queuedFetch rather than sent back on the callback.

mitra42 commented 5 years ago

Fixed that but now need to

mitra42 commented 5 years ago

Fixed - and pushed to github and npm