TypeError: Cannot set property 'userId' of undefined about lireddit HOT 3 OPEN

If you hover into the session from request object, you'll get:

This request's Session object. Even though this property isn't marked as optional, it won't exist until you use the express-session middleware Declaration merging can be used to add your own properties.

The default Session object isn't able to add out custom fields like UserId. My take was to do declaration merging by:
First, updating the tsconfig file to use the declaration i.e, adding up

  "typeRoots": [
    "./src/types", // for express.session declaration merging 
    "./node_modules/@types" 
  ]

And adding this in the types.ts:

import "express-session";

declare module 'express-session' {
    export interface Session {
        userId: number;
    }
}
export type myContext = {
    em: EntityManager<IDatabaseDriver<Connection>>;
    req: Request & { session: Session & Partial<SessionData> };
    res: Response;
}

Also, see check the secure property in the cookie property of app.use(session(...)). I had to set it to false to make the cookie work. Hope this helps.

from lireddit.

I first solve this problem with @akumaaayush's approach, and then, I found an issue that is all about this problem, so I came to share it.
#13

from lireddit.

@akumaaayush @ryankert01 thank you for your help but I have tried all this plus everything in #13 and I am still getting the same error.

from lireddit.