Cannot read property '_destroySSL' of undefined at destroySSL about lighthouse

[paulirish]

@two7sclash evaluate this after lighthouse is required:

self.setImmediate = function(callback, ...argsForCallback) {
  Promise.resolve().then(() => callback(...argsForCallback));
  return 0;
};

sorry for this, guys.

[mohsen1]

Still seeing this in TravisCI when I launch chrome programmatically

[paulirish]

Just ran into this problem again, while working on pwmetrics.. here's what I found out:

the _destroySSL stuff is happening inside _tls_wrap, which the request module uses:

As you can see, the this is passed as a parameter to setImmediate but a heartbeat later, that param isn't there. And why not?

chrome-devtools-frontend has been polyfilling setImmediate, and unfortunately for us... incorrectly:

:)

---

Update: I'm submitting a patch to fix it in devtools

[gauntface]

Looks like:

TypeError: Cannot read property '_destroySSL' of undefined
at destroySSL (_tls_wrap.js:370:7)

is coming from the chrome remote interface close() call. Not sure there is anything that can be done to silence the error

[paulirish]

I tried to diagnose this exception but only got so far. I'm pretty sure it's a node core bug, as that area (_tls_wrap.js) has gotten some fixes.

[paulirish]

TypeError: Cannot read property '_destroySSL' of undefined at destroySSL (_tls_wrap.js:370:7)

Took another stab last night. AFAIK, this is due to the request call for the manifest, and not cri's close(). 😌 So we'll be done with it soon.

[paulirish]

Broke out the other two issues into #106 and #105.

Retitled this to be just the destroySSL. This will be fixed when #83 is done. \o/

[paulirish]

fixed by #83

[ebidel]

I just ran into this using lighthouse 1.4.1 as a submodule. @paulirish did you ever submit a CL for devtools?

[paulirish]

Yes. Fixed over there: https://codereview.chromium.org/2615083002

But we'll need to bump our dependency to handle the update: #1535

[two7sclash-zz]

still seeing this too, running lighthouse on heroku.