Cannot read property 'cookie' of undefined

[expressjs]

Cannot read property 'cookie' of undefined

[dougwilson]

This middleware is for parsing HTTP requests; the stack trace is coming from where socket.io is passing a socket object. This middleware is not a socket.io middleware, which is why you are having an issue.

[danpe]

I'm also getting this 👍

[Stofkn]

My code to make it this cookie-parser module work together with socket.io:

// ### Cookie parser

// Wrapper arround Express cookie parser, so we can use the same cookie parser for socket.io.
// Parse Cookie header and populate `socket.request.cookies` with an object keyed by the cookie names.
// Uses signed cookies by passing a secret string, which assigns `socket.request.secret` so it may be used by other middleware.

function cookieParserWrapper (socket, next) {
  // request, response and callback
  cookieParser(socket.request, {}, next);
}`

I hope it will help someone else!

Kind regards
Kristof

[dougwilson]

I don't know how to use socket.io, but this module requires an HTTP request as the first argument, and node.js core itself adds a headers property. If you are getting this error, you are simply using this module incorrectly with socket.io, because you are using it somewhere that is not giving it a HTTP request.

[rmethlie]

The cookie-parser reference you pass as a passportSocketio.authorize({...}) configuration attribute will get the app crypto secret when it is invoked, not the request instance you'd expect. Therefore, do not create an instance of the cookie-parser. Instead simply get a reference to the library and pass that in.

So don't do this

app.io.use(passportSocketIO.authorize({
    passport: passport,
    cookieParser: require('cookie-parser')(config.cryptoKey),
    ...

That results in the error you are seeing because the cookie-parser will now expect a request as an argument. Instead do this:

app.io.use(passportSocketIO.authorize({
    passport: passport,
    cookieParser: require('cookie-parser'), // dont init, let passport.socketio do that for you

Hope that helps.