-
Notifications
You must be signed in to change notification settings - Fork 24
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
Expose req
to genid
function similar to express-session
#372
base: master
Are you sure you want to change the base?
Conversation
|
Codecov Report
@@ Coverage Diff @@
## master #372 +/- ##
=======================================
Coverage 99.17% 99.17%
=======================================
Files 6 6
Lines 121 121
Branches 37 36 -1
=======================================
Hits 120 120
Misses 1 1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
req
to genid
function similar to express-session
Any update on this? Our current project could use this feature. |
It's more than a year since the last commit on this repo, and I'm afraid if it is being maintained anymore. Meanwhile, you can use my fork which includes the changes in this PR: @zignis/next-session |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm. I can release a new version once this is merged
Pass the request object to the id generator function as done by express-session, to allow generating session ids based off of some data in the request.