Skip to content
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

feat: Add Session.regenerate() method (#34) #221

Merged
merged 1 commit into from
Feb 4, 2023

Conversation

lehni
Copy link
Contributor

@lehni lehni commented Dec 5, 2022

And make both save() and regenerate() compatible with koa-passport v5.0.0

This is based on #74 which is a bit outdated, and more work was needed to make this work with koa-passport due to the use of callback arguments.

See rkusa/koa-passport#181 for more details.

And make both save() and regenerate() compatible with koa-passport
@ehedges03
Copy link

I would really like to see this added as well.

I have tested the fork of @lehni and have run into no issues.

I hope this can be merged soon.

@lehni
Copy link
Contributor Author

lehni commented Feb 4, 2023

@fengmk2 I am sorry to nag, but is there any chance to consider this? the latest version of koa-passport doesn't work without these methods. I am happy to address any concerns you might have.

@fengmk2 fengmk2 self-assigned this Feb 4, 2023
@fengmk2
Copy link
Member

fengmk2 commented Feb 4, 2023

I will merge this change after setup github action.

@fengmk2 fengmk2 changed the title Add Session.regenerate() method (#34) feat: Add Session.regenerate() method (#34) Feb 4, 2023
@fengmk2 fengmk2 merged commit 4cd3bef into koajs:master Feb 4, 2023
@fengmk2
Copy link
Member

fengmk2 commented Feb 4, 2023

6.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants