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

Calling User.save() removes all current access tokens even not change user email or password. #3124

Closed
iicy90 opened this issue Jan 20, 2017 · 2 comments
Assignees
Labels

Comments

@iicy90
Copy link

iicy90 commented Jan 20, 2017

#3021
in common/models/user.js:

var isFullReplaceChangingPassword = !!ctx.instance;
// ...
ctx.hookState.isPasswordChange = isPartialUpdateChangingPassword || 
  isFullReplaceChangingPassword;`

ctx.instance always true when you do User.save()

@davidcheung
Copy link
Contributor

@icy90 I see. Please open a new github issue for this problem, so that we can move the discussion there, and mention my handle there so that I get email notification.

@bajtos

@bajtos
Copy link
Member

bajtos commented Feb 6, 2017

Fixed.

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

No branches or pull requests

3 participants