Cory Virok
Cory Virok
I'm also seeing the same behavior when attempting to read the cookie from an APIRoute. This is breaking the SSR example. I'd recommend bumping this up to a p1 since...
Upgrading to Node 18.9 (from 18.2) seems to fix it. Also, more info here: https://discord.com/channels/830184174198718474/1004159859131944981/1004464449601667154
Thanks for starting the convo, Brian. I would probably expose a new configuration option that would allow the user to provide a function that would be used instead of `__repr__()`...
Thanks for the review, @dummdidumm, @Conduitry and @benmccann. I updated the code style to use `snake_case` for internal vars. I couldn't find any unit tests, so I added a test...
@Conduitry anything blocking this from being merged?
@benmccann anything else needed before merging? (I'm just trying to tidy up loose PR ends :)
> Looks good to me, but you will need someone more familiar with the code like Li Hau or Conduitry to sign-off on it Gotcha, thanks.
Seems like a good opportunity to deprecate old Node versions and for a major version bump in this module. Since this basically runs in every single Express app, I'm sure...
Yikes... some of the dependencies have high severity security issues. More cause for upgrades and modernization. 
@jondeandres what are the next steps for this? Maybe @rokob can take it over?