Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

this.pingIntervalTimer.refresh is not a function

See original GitHub issue

Describe the bug socket.js:108 Uncaught TypeError: this.pingIntervalTimer.refresh is not a function at Socket.onPacket (socket.js:108)

To Reproduce engine.io: 6.1.0 engine.io-client: 6.1.1 socket.io: 4.4.0 socket.io-client: 4.4.0

Platform:

OS: windows 10

Additional context Adding console.log(typeof this.pingIntervalTimer) just before line 108 shows that it’s a number, not a Timer.

this.pingIntervalTime.refresh() was added here it seems: https://github.com/socketio/engine.io/commit/37474c7e67be7c5f25f9ca2d4ea99f3a256bd2de

Issue Analytics

State:
Created 2 years ago
Comments:8 (8 by maintainers)

Top GitHub Comments

1reaction

JeffWindercommented, Dec 8, 2021

I think it would be best to explicitly import setTimeout and clearTimeout from Node’s timers module, by adding this in lib/socket.ts: import { setTimeout, clearTimeout } from "timers"; That would make sure compilers don’t mistaken those functions for javascript timers, of which setTimeout doesn’t have a refresh function. Hence the error in my scenario.

I can also make a PR for that tomorrow if you like.

0reactions

darrachequesnecommented, Dec 14, 2021

Closed by https://github.com/socketio/engine.io/pull/632.