TypeError: Cannot set property 'userId' of undefined

[dreeverl20]

Hi Ben,

Thank you so much for this tutorial.

If any one can help me I would really appreciate it

I have been stuck on this part for over two days. Any time I add in line 141 I get the error below in my sandbox . I have also added in a screen shot from my types file. I also do not see any cookie being set. I f I remove line 141 I do not get an error it logs me in but still does not set a cookie.

Thanks for any help!

)

[akumaaayush]

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.

[ryankert01]

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

[dreeverl20]

@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.