Skip to content

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

Merged
fengmk2 merged 1 commit into
koajs:masterfrom
ditojs:feature/regeneration
Feb 4, 2023
Merged

feat: Add Session.regenerate() method (#34)#221
fengmk2 merged 1 commit into
koajs:masterfrom
ditojs:feature/regeneration

Conversation

@lehni

@lehni lehni commented Dec 5, 2022

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

lehni commented Feb 4, 2023

Copy link
Copy Markdown
Contributor Author

@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

fengmk2 commented Feb 4, 2023

Copy link
Copy Markdown
Member

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

fengmk2 commented Feb 4, 2023

Copy link
Copy Markdown
Member

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