Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Exception: Cannot destructure property 'socket' of '(intermediate value)(intermediate value)(intermediate value)' as it is undefined. #8

Open
danfuzz opened this issue Feb 10, 2023 · 2 comments

Comments

@danfuzz
Copy link

danfuzz commented Feb 10, 2023

Not sure what caused this, but I figured I'd at least drop a note here in case it helps…

I just saw my server die with this exception:

          "message": "Cannot destructure property 'socket' of '(intermediate value)(intermediate value)(intermediate value)' as it is undefined.",
          "stack": [
            {
              "name": "Http2SecureServer.app",
              "file": "/home/milk-user/lib/lactoserv/lib/node_modules/http2-express-bridge/lib/bridge.js",
              "line": 52,
              "col": 9
            },
            {
              "name": "Http2SecureServer.emit",
              "file": "node:events",
              "line": 537,
              "col": 28
            },
            {
              "name": "Http2SecureServer.onServerStream",
              "file": "node:internal/http2/compat",
              "line": 883,
              "col": 10
            },
...

(The original error stack was serialized/deserialized.)

The cited line corresponds to this in the original (unprocessed) source, in bridge.js:

    const app = function(req, res, next) {
        const { socket } = req.httpVersion === '2.0' ?
                                req.stream.session : req;

        //Checking alpnProtocol for http2
        app.handle(req, res, next);
    };

Somewhat curiously, it looks like that socket value isn't even used, so maybe there's no point to the line at all?

Anyway, hope this helps.

@victor-homyakov
Copy link

Bump. Is there any progress on this issue?

@danfuzz
Copy link
Author

danfuzz commented Jul 17, 2024

Pretty sure this project is abandoned by its owner at this point.

FWIW I ended up dropping this project from my dependencies in favor of writing my own adapter… and then ultimately I dropped Express entirely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants